Actually CellTreeNodeView is still work in progress and won't be submitted as part of the current CL. Sorry for the confusion.
On Mon, Aug 2, 2010 at 12:14 PM, <jlaba...@google.com> wrote: > LGTM > > > http://gwt-code-reviews.appspot.com/710802/diff/1013/10009 > File user/src/com/google/gwt/user/cellview/client/CellTreeNodeView.java > (right): > > http://gwt-code-reviews.appspot.com/710802/diff/1013/10009#newcode536 > user/src/com/google/gwt/user/cellview/client/CellTreeNodeView.java:536: > Extra space. > > http://gwt-code-reviews.appspot.com/710802/diff/1013/10009#newcode564 > user/src/com/google/gwt/user/cellview/client/CellTreeNodeView.java:564: > System.out.println("keyboardOpen"); > Remove SOP > > http://gwt-code-reviews.appspot.com/710802/diff/1013/10009#newcode566 > user/src/com/google/gwt/user/cellview/client/CellTreeNodeView.java:566: > // if (isOpen()) { > Remove comments or add a TODO. > > http://gwt-code-reviews.appspot.com/710802/diff/1013/10009#newcode572 > user/src/com/google/gwt/user/cellview/client/CellTreeNodeView.java:572: > System.out.println("keyboardUp"); > Remove SOP > > http://gwt-code-reviews.appspot.com/710802/diff/1013/10009#newcode575 > user/src/com/google/gwt/user/cellview/client/CellTreeNodeView.java:575: > if (prev == keyboardSelectedElement) { > Add a TODO to move focus to the parent node. > > > http://gwt-code-reviews.appspot.com/710802/show > -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
