Closed Bug 810364 Opened 8 years ago Closed 8 years ago

[AccessFu] Record starting touch coordinate in gesture detection

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla19

People

(Reporter: eeejay, Assigned: eeejay)

Details

Attachments

(1 file)

No description provided.
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+
https://hg.mozilla.org/mozilla-central/rev/908a6c66f03d
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
You need to log in before you can comment on or make changes to this bug.