http://gwt-code-reviews.appspot.com/1708803/diff/1/user/src/com/google/gwt/cell/client/EditTextCell.java File user/src/com/google/gwt/cell/client/EditTextCell.java (right):
http://gwt-code-reviews.appspot.com/1708803/diff/1/user/src/com/google/gwt/cell/client/EditTextCell.java#newcode222 user/src/com/google/gwt/cell/client/EditTextCell.java:222: } else if (value != null && value.length() > 0) { On 2012/05/10 19:14:52, tbroyer wrote:
What if value is made of spaces? Does it work? Doesn't it needs a
too? I don't want everyone to pay the penalty of a regex or trim() for an relatively obscure case (all whitespace). The real answer is that Cells should be able to receive events triggered on the TD element. That could be an option in CellTable, but I don't have time to add it at the moment. http://gwt-code-reviews.appspot.com/1708803/diff/1/user/src/com/google/gwt/cell/client/EditTextCell.java#newcode229 user/src/com/google/gwt/cell/client/EditTextCell.java:229: sb.appendHtmlConstant(" "); On 2012/05/10 19:14:52, tbroyer wrote:
How about using \u00A0 instead? Would save a few bytes in the compiled
code. I can switch to \u00A0. http://gwt-code-reviews.appspot.com/1708803/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
