Lior Vernia has posted comments on this change. Change subject: webadmin: Focus on label widget by default ......................................................................
Patch Set 1: (2 comments) http://gerrit.ovirt.org/#/c/32096/1/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/AddRemoveRowWidget.java File frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/AddRemoveRowWidget.java: Line 310: @Override Line 311: public void focusInput() { Line 312: super.focusInput(); Line 313: Line 314: ListIterator<Pair<T, V>> last = items.listIterator(items.size()); > I think the following code is more readable- Left as is, no functional difference... Line 315: if (last.hasPrevious()) { Line 316: V widget = last.previous().getSecond(); Line 317: if (widget instanceof Focusable) { Line 318: ((Focusable) widget).setFocus(true); http://gerrit.ovirt.org/#/c/32096/1/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/generic/ListModelSuggestBox.java File frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/generic/ListModelSuggestBox.java: Line 27: Scheduler.get().scheduleDeferred(new ScheduledCommand() { Line 28: Line 29: @Override Line 30: public void execute() { Line 31: asSuggestBox().showSuggestionList(); > Please format Done Line 32: } Line 33: }); Line 34: } Line 35: }); -- To view, visit http://gerrit.ovirt.org/32096 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I36c9ae0b7ac9d61112ace1829434c9c685b85a22 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <[email protected]> Gerrit-Reviewer: Alona Kaplan <[email protected]> Gerrit-Reviewer: Lior Vernia <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
