[AccessFu] Record starting touch coordinate in gesture detection

RESOLVED FIXED in mozilla19

Status

()

Core
Disability Access APIs
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: eeejay, Assigned: eeejay)

Tracking

Trunk
mozilla19
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

5 years ago
Created attachment 680111 [details] [diff] [review]
Use startX as current X in touch adapter
Attachment #680111 - Flags: review?(dbolter)
Comment on attachment 680111 [details] [diff] [review]
Use startX as current X in touch adapter

Review of attachment 680111 [details] [diff] [review]:
-----------------------------------------------------------------

::: accessible/src/jsat/TouchAdapter.jsm
@@ +207,5 @@
>   * of one single touch.
>   */
>  function TouchPoint(aTouch, aTime, aDPI) {
> +  this.startX = this.x = aTouch.screenX;
> +  this.startY = this.y = aTouch.screenY;

You can probably remove this check in the update function:
if (lastX != undefined && lastY != undefined)

(And possibly other cleanup)
Attachment #680111 - Flags: review?(dbolter) → review+
(Assignee)

Comment 3

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/908a6c66f03d
Assignee: nobody → eitan

Comment 4

5 years ago
https://hg.mozilla.org/mozilla-central/rev/908a6c66f03d
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
You need to log in before you can comment on or make changes to this bug.