Alona Kaplan has uploaded a new change for review. Change subject: webadmin: adjust sub menu popup location to fit screen + RTL popup support ......................................................................
webadmin: adjust sub menu popup location to fit screen + RTL popup support no bz If the subMenu popup doesn't fit the screen it will be opened in another position if poosible, or with a scroll bar. This feature is a workaround to the problem introduced in http://gerrit.ovirt.org/#change,2489 It should have been implemented in the gwt MenuBar. But it wasn't. See- http://code.google.com/p/google-web-toolkit/issues/detail?id=407 http://code.google.com/p/google-web-toolkit/issues/detail?id=3924 In order to implement this workaround I had to use gwt MenuBar private method. I did it with the help of JSNI. Change-Id: Ide1b780d9db245f61aa7332a50cf56b3a4761065 --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/utils/PopupUtils.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/MenuBar.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/PopupPanel.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/action/AbstractActionPanel.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/action/MenuPanelPopup.java 5 files changed, 261 insertions(+), 67 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/73/2673/1 -- To view, visit http://gerrit.ovirt.org/2673 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ide1b780d9db245f61aa7332a50cf56b3a4761065 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
