Matthew Dempsky has posted comments on this change.
Change subject: Workaround for resolving mouse button on mouse move in
Firefox
......................................................................
Patch Set 2:
(2 comments)
Idea seems okay to me. Just some style nits.
....................................................
File user/src/com/google/gwt/dom/client/DOMImplMozilla.java
Line 144: if (evt.getType().equals((BrowserEvents.MOUSEMOVE))) {
No need for the extra parentheses around BrowserEvents.MOUSEMOVE.
Line 294: if(evt.buttons){
Fix the spacing here (and below) please. It should be "if (evt.buttons) {".
--
To view, visit https://gwt-review.googlesource.com/2330
To unsubscribe, visit https://gwt-review.googlesource.com/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib73997af56ce0e7da5b41814a7ac2b208ab022aa
Gerrit-PatchSet: 2
Gerrit-Project: gwt
Gerrit-Branch: master
Gerrit-Owner: John Ahlroos <[email protected]>
Gerrit-Reviewer: Matthew Dempsky <[email protected]>
Gerrit-HasComments: Yes
--
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
---
You received this message because you are subscribed to the Google Groups "Google Web Toolkit Contributors" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
