Einav Cohen has posted comments on this change. Change subject: webadmin: Control refresh behavior (#802378) ......................................................................
Patch Set 1: Vojtech, I agree with Gilad on this: This check-box seems strange, I am not sure if people would understand what it is used for and/or ever un-check it. We should keep in mind what is good for a typical use case (automated GUI tests (Selenium) launching multiple browsers in parallel on same machine is not a typical use case). I suggest to remove the check-box and have the refresh rate set to the minimal one when out of focus. In the future, we can consider having all focus-refresh-rates and out-of-focus-refresh-rate configurable. [I will also put this note in the bugzilla] -- To view, visit http://gerrit.ovirt.org/3197 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I08c6dd9fe2c63d58b7a2d865552564d7e73425c2 Gerrit-PatchSet: 1 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
