Vojtech Szocs has posted comments on this change.

Change subject: webadmin: Update Firefox supported version
......................................................................


Patch Set 2:

Changed IE9 support to "IE9 and above". As for Chrome, we can add it later, but 
I think we should test that everything looks as expected in Chrome, before 
adding it as "supported browser".

Adding OS as "supported browser" parameter sounds good, but I suggest to do it 
in another patch, since it would also affect other parts (message template and 
application code that calls it).

As for "browser not supported" message within main section (after user logs 
in), this patch removes it, as Gilad says we already have a "browser not 
supported" message in login section.

I'll uload an updated patch soon.

--
To view, visit http://gerrit.ovirt.org/3137
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5fe7dda86294a0850243ce08f19b25f7b25b647f
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vojtech Szocs <vsz...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Einav Cohen <eco...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to