Thomas Broyer has posted comments on this change.

Change subject: Fix ValuePicker.setValue(T) to no longer fire events.
......................................................................


Patch Set 2:

AFAICT, no, they won't be "in sync", as it's the purpose of the SelectionChangeHandler to update the current value (this.value) with the one being selected (using the mouse or keyboard) in the CellList. This is specifically where we want to fire a ValueChangeEvent.

--
To view, visit https://gwt-review.googlesource.com/2980
To unsubscribe, visit https://gwt-review.googlesource.com/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I20042df6fa0770e4505b1acae8dc28aa77ab7e3a
Gerrit-PatchSet: 2
Gerrit-Project: gwt
Gerrit-Branch: master
Gerrit-Owner: Thomas Broyer <t.bro...@gmail.com>
Gerrit-Reviewer: Jens Nehlmeier <jens.nehlme...@gmail.com>
Gerrit-Reviewer: Thomas Broyer <t.bro...@gmail.com>
Gerrit-HasComments: No

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
--- You received this message because you are subscribed to the Google Groups "GWT Contributors" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to google-web-toolkit-contributors+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to