Greg Sheremeta has uploaded a new change for review.
Change subject: userportal, webadmin: added TODO-GWT to BookmarkList
......................................................................
userportal, webadmin: added TODO-GWT to BookmarkList
BookmarkList mentioned a GWT bug, but did not have a TODO-GWT marker.
Added the comment.
No functional changes -- just a comment line.
Change-Id: I5bc03964ede462878fd8df89ea67d6282dfb11aa
Signed-off-by: Greg Sheremeta <gsher...@redhat.com>
---
M
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/bookmark/BookmarkList.java
1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/32/38732/1
diff --git
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/bookmark/BookmarkList.java
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/bookmark/BookmarkList.java
index 0caafa6..074accf 100644
---
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/bookmark/BookmarkList.java
+++
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/bookmark/BookmarkList.java
@@ -64,6 +64,7 @@
// Using KeyboardSelectionPolicy.BOUND_TO_SELECTION is preferable, but
broken (see
// gwt issue 6310). Instead, use ENABLED and handle the keyboard
input ourselves.
+ // TODO-GWT is this issue now fixed?
display.addDomHandler(new KeyDownHandler() {
@Override
@SuppressWarnings("unchecked")
--
To view, visit https://gerrit.ovirt.org/38732
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5bc03964ede462878fd8df89ea67d6282dfb11aa
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Greg Sheremeta <gsher...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
