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

Reply via email to