Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(248)

Issue 5150047: Add touch handling to BrowserEvent, removing from Dragger

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 5 months ago by Andrew Mattie
Modified:
11 years, 6 months ago
Reviewers:
nnaze
Base URL:
http://closure-library.googlecode.com/svn/trunk/
Visibility:
Public.

Description

As reported here[1], the dragger implementation for touch events is broken in that it doesn't act on the actual event but rather on the relevant touch. The problem prevents methods like preventDefault and stopPropagation from working and masks the touch event when it's dispatched in the DragEvent. This patch removes the touch event handling functionality from dragger and instead moves it into BrowserEvent. http://code.google.com/p/closure-library/issues/detail?id=382 [1] https://groups.google.com/d/topic/closure-library-discuss/0MO-sSKN2YA/discussion

Patch Set 1 #

Patch Set 2 : Resolved conflicts with SVN r1562 #

Total comments: 12

Patch Set 3 : Styling and consistency fixes #

Patch Set 4 : Updated for compatibility w/ SVN r1780 #

Patch Set 5 : Updated for compatibility w/ SVN r2127 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+99 lines, -46 lines) Patch
closure/goog/events/browserevent.js View 1 2 3 4 2 chunks +32 lines, -11 lines 0 comments Download
closure/goog/events/browserevent_test.html View 1 2 3 4 14 chunks +67 lines, -13 lines 0 comments Download
closure/goog/fx/dragger.js View 1 2 3 4 4 chunks +0 lines, -22 lines 0 comments Download

Messages

Total messages: 9
nnaze
http://codereview.appspot.com/5150047/diff/3002/closure/goog/events/browserevent.js File closure/goog/events/browserevent.js (right): http://codereview.appspot.com/5150047/diff/3002/closure/goog/events/browserevent.js#newcode245 closure/goog/events/browserevent.js:245: var relevantTouch; var touch; http://codereview.appspot.com/5150047/diff/3002/closure/goog/events/browserevent.js#newcode256 closure/goog/events/browserevent.js:256: if (!!relevantTouch) drop ...
12 years, 1 month ago (2012-02-06 23:54:48 UTC) #1
Andrew Mattie
http://codereview.appspot.com/5150047/diff/3002/closure/goog/events/browserevent.js File closure/goog/events/browserevent.js (right): http://codereview.appspot.com/5150047/diff/3002/closure/goog/events/browserevent.js#newcode245 closure/goog/events/browserevent.js:245: var relevantTouch; On 2012/02/06 23:54:48, nnaze wrote: > var ...
12 years, 1 month ago (2012-02-07 06:50:18 UTC) #2
Andrew Mattie
Ping. On 2012/02/07 06:50:18, Andrew Mattie wrote: > http://codereview.appspot.com/5150047
12 years ago (2012-02-28 16:53:02 UTC) #3
chromaticbum
On 2012/02/28 16:53:02, Andrew Mattie wrote: > Ping. > > On 2012/02/07 06:50:18, Andrew Mattie ...
12 years ago (2012-03-19 20:10:45 UTC) #4
Andrew Mattie
Which hunk was rejected? Were you using patch set 3? I was able to download ...
12 years ago (2012-03-19 20:21:03 UTC) #5
chromaticbum
Ah, I see. I was using patch set 1. Tohught I had to incrementally apply ...
12 years ago (2012-03-19 21:40:12 UTC) #6
chromaticbum
Patch set 3 works fine. On 2012/03/19 21:40:12, chromaticbum wrote: > Ah, I see. > ...
12 years ago (2012-03-19 21:46:43 UTC) #7
Andrew Mattie
Updated for compatibility w/ SVN r1780
11 years, 11 months ago (2012-04-25 19:09:36 UTC) #8
Andrew Mattie
11 years, 6 months ago (2012-08-27 21:02:46 UTC) #9
Updated for compatibility w/ SVN r2127
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b