http://gwt-code-reviews.appspot.com/906801/diff/1/12 File samples/showcase/src/com/google/gwt/sample/showcase/client/content/cell/CwCellValidation.java (right):
http://gwt-code-reviews.appspot.com/906801/diff/1/12#newcode104 samples/showcase/src/com/google/gwt/sample/showcase/client/content/cell/CwCellValidation.java:104: input.getStyle().setColor("blue"); CwCellValidation is an example in Showcase, so we don't need to support user customization. http://gwt-code-reviews.appspot.com/906801/diff/1/13 File user/src/com/google/gwt/cell/client/AbstractCell.java (right): http://gwt-code-reviews.appspot.com/906801/diff/1/13#newcode128 user/src/com/google/gwt/cell/client/AbstractCell.java:128: * Called when the user triggers a <code>keydown</code> event On 2010/09/21 21:08:31, rice wrote:
with the ENTER key
Done. http://gwt-code-reviews.appspot.com/906801/diff/1/14 File user/src/com/google/gwt/cell/client/AbstractEditableCell.java (right): http://gwt-code-reviews.appspot.com/906801/diff/1/14#newcode86 user/src/com/google/gwt/cell/client/AbstractEditableCell.java:86: * may chooses to pass keystrokes directly to the cell rather than using them On 2010/09/21 21:08:31, rice wrote:
chooses -> choose
Done. http://gwt-code-reviews.appspot.com/906801/diff/1/15 File user/src/com/google/gwt/cell/client/AbstractInputCell.java (right): http://gwt-code-reviews.appspot.com/906801/diff/1/15#newcode36 user/src/com/google/gwt/cell/client/AbstractInputCell.java:36: private final Set<String> consumedEvents = new HashSet<String>(); On 2010/09/21 21:08:31, rice wrote:
AbstractCell already has a final Set<String> consumedEvents
Done. http://gwt-code-reviews.appspot.com/906801/diff/1/15#newcode50 user/src/com/google/gwt/cell/client/AbstractInputCell.java:50: super(); Done. Defer to a static method to add the additional events. http://gwt-code-reviews.appspot.com/906801/diff/1/15#newcode64 user/src/com/google/gwt/cell/client/AbstractInputCell.java:64: super(); On 2010/09/21 21:08:31, rice wrote:
Change to super(consumedEvents) and delete the addAll
Done. http://gwt-code-reviews.appspot.com/906801/diff/1/15#newcode70 user/src/com/google/gwt/cell/client/AbstractInputCell.java:70: public Set<String> getConsumedEvents() { On 2010/09/21 21:08:31, rice wrote:
Delete, implemented in AbstractCell
Done. http://gwt-code-reviews.appspot.com/906801/diff/1/15#newcode144 user/src/com/google/gwt/cell/client/AbstractInputCell.java:144: private void init() { getConsumedEvents() isn't really meant to be mutable. Instead, we now add the events in a static method and pass them to the super constructor. http://gwt-code-reviews.appspot.com/906801/diff/1/16 File user/src/com/google/gwt/cell/client/ActionCell.java (right): http://gwt-code-reviews.appspot.com/906801/diff/1/16#newcode78 user/src/com/google/gwt/cell/client/ActionCell.java:78: onEnterKeyDown(parent, value, key, event, valueUpdater); The "other action" varies depending on the Cell, but ENTER always puts it into edit mode if the cell supports editing. And I can't think of a better name. http://gwt-code-reviews.appspot.com/906801/diff/1/21 File user/src/com/google/gwt/cell/client/CompositeCell.java (right): http://gwt-code-reviews.appspot.com/906801/diff/1/21#newcode151 user/src/com/google/gwt/cell/client/CompositeCell.java:151: // The first child that takes focus ones. One one child should ever be in On 2010/09/21 21:08:31, rice wrote:
ones -> wins?
Done. http://gwt-code-reviews.appspot.com/906801/diff/1/40 File user/src/com/google/gwt/user/cellview/client/CellTree.java (right): http://gwt-code-reviews.appspot.com/906801/diff/1/40#newcode682 user/src/com/google/gwt/user/cellview/client/CellTree.java:682: public void setAccessKey(char key) { On 2010/09/21 21:08:31, rice wrote:
Document that -1 means no access key
Done. http://gwt-code-reviews.appspot.com/906801/diff/1/40#newcode808 user/src/com/google/gwt/user/cellview/client/CellTree.java:808: if (KeyboardSelectionPolicy.DISABLED == getKeyboardSelectionPolicy()) { On 2010/09/21 21:08:31, rice wrote:
Maybe create a method boolean isKeyboardSelectionDisabled() in a
suitable
superclass?
Done. http://gwt-code-reviews.appspot.com/906801/diff/1/45 File user/src/com/google/gwt/user/cellview/client/HasKeyboardSelectionPolicy.java (right): http://gwt-code-reviews.appspot.com/906801/diff/1/45#newcode42 user/src/com/google/gwt/user/cellview/client/HasKeyboardSelectionPolicy.java:42: BOUND On 2010/09/21 21:08:31, rice wrote:
Maybe BOUND_TO_SELECTION?
Done. http://gwt-code-reviews.appspot.com/906801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
