Change in ovirt-engine[master]: userportal, webadmin: Unable to scroll down template list usi...
tjelinek at redhat.com
tjelinek at redhat.com
Thu Jul 10 06:31:17 UTC 2014
Tomas Jelinek has submitted this change and it was merged.
Change subject: userportal,webadmin: Unable to scroll down template list using IE9
......................................................................
userportal,webadmin: Unable to scroll down template list using IE9
Three issues:
1: when clicking the scrollbar next to the template selection, IE9 did not fire
a click event properly (a known IE9 bug). Fixed by attaching the listeners to
the parent element (the whole suggestion box).
2: when using up/down keys, the proper template has been selected, but the view
was not scrolled into it. Fixed by scrolling into the view after pressing
arrow keys and also when opening the suggestions.
3: tweaked the CSS a bit so the content of the suggestions are not shifted
Tested on IE9 and FF.
Change-Id: I93312c75f7605cb007ae395d900ef8ced35c6b14
Bug-Url: https://bugzilla.redhat.com/1100194
Signed-off-by: Tomas Jelinek <tjelinek at redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/BaseListModelSuggestBox.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/ListModelTypeAheadListBox.java
2 files changed, 52 insertions(+), 4 deletions(-)
Approvals:
Tomas Jelinek: Verified
Alexander Wels: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/28939
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I93312c75f7605cb007ae395d900ef8ced35c6b14
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the Engine-commits
mailing list