Ticket #3208 (closed feature: fixed)

Opened 2 years ago

Last modified 2 years ago

some refactoring in the touch-specific code of draw handlers

Reported by: erilem Owned by: tschaub
Priority: minor Milestone: 2.11 Release
Component: Handler.Point Version: 2.10
Keywords: mobile Cc:
State: Commit

Description

This ticket suggests refactoring the touch-specific code of the drawing handlers a bit. More specifically:

  • move the doubletap handling code from the point handler to the path handler, doubletap indeed doesn't pertain to point drawing
  • remove the isDblclick flag, which is no longer needed with #3202 now closed.

Attachments

patch-3208-A0.diff Download (5.0 KB) - added by erilem 2 years ago.

Change History

Changed 2 years ago by erilem

See patch-3208-A0.diff Download. Tests pass in IE8, FF3, and Chromium10. Please review.

Changed 2 years ago by erilem

  • state set to Review

Changed 2 years ago by erilem

Changed 2 years ago by sbrunner

  • state changed from Review to Commit

OK for me -> commit !

CU Stéphane

Changed 2 years ago by erilem

  • status changed from new to closed
  • resolution set to fixed

(In [11773]) some refactoring in the touch-specific code of draw handlers, r=sbrunner (closes #3208)

Note: See TracTickets for help on using tickets.