Reviewers: rchandia,

Description:
Fixing a few bugs in Cell widgets.
Issue 5625: Fixes a bug in CellBrowser where the first item in each list
is not clickable because we are too optimistic about not reselecting a
row that is already selected. Now we will reselect a row if the row
index changed OR if there is no row value selected, which ensures that
the 0th row can be selected.
Issue 5632: Fixes a bug where you have to click an EditTextCell twice to
switch it to edit mode. The bug is actually that we are stealing focus
away from the cell even though it is in edit mode.
Fixes an NPE in EditTextCell when ValueUpdater is null.
Fixes a bug in CellList where the key is not passed to Cell#render().


Please review this at http://gwt-code-reviews.appspot.com/1141801/show

Affected files:
  M user/src/com/google/gwt/cell/client/EditTextCell.java
  M user/src/com/google/gwt/user/cellview/client/CellBrowser.java
  M user/src/com/google/gwt/user/cellview/client/CellList.java
  M user/src/com/google/gwt/user/cellview/client/CellTable.java
  M user/src/com/google/gwt/user/cellview/client/CellTreeNodeView.java
  M user/src/com/google/gwt/user/cellview/client/HasDataPresenter.java
  M user/test/com/google/gwt/user/cellview/client/CellListTest.java
  M user/test/com/google/gwt/user/cellview/client/HasDataPresenterTest.java


--
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to