A previous change broke some backward-compatibility with RowHoverEvent (issue1637803)

6 views
Skip to first unread message

is...@google.com

unread,
Feb 3, 2012, 2:54:49 PM2/3/12
to jlab...@google.com, google-web-tool...@googlegroups.com, re...@gwt-code-reviews-hr.appspotmail.com
Reviewers: jlabanca,

Description:
A previous change broke some backward-compatibility with RowHoverEvent
(events did not fire when expected). This change fixes the compatibility
issue, clarifies the event's scope and introduces a new field in the
event
(hovering scope).

Clients can use this field to differentiate between hover events on the
same
row and hovering between different rows, allowing more control on the
event's
processing.

Review by: pengz...@google.com

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

Affected files:
M user/src/com/google/gwt/user/cellview/client/AbstractCellTable.java
M user/src/com/google/gwt/user/cellview/client/RowHoverEvent.java


jlab...@google.com

unread,
Feb 3, 2012, 2:59:31 PM2/3/12
to is...@google.com, google-web-tool...@googlegroups.com, re...@gwt-code-reviews-hr.appspotmail.com
Reply all
Reply to author
Forward
0 new messages