#15188 closed defect (fixed)
dojox/gesture/tap sometimes fire exceptions
| Reported by: | cjolif | Owned by: | Evan |
|---|---|---|---|
| Priority: | undecided | Milestone: | 1.8 |
| Component: | Dojox | Version: | 1.7.0 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description
This was found on trunk. I don't know if the problem exist in 1.7.x.
When you register a tap event, and you touch one finger and then quickly a second one on the screen _without_ removing the fingers from the screen you get an exception in _isTap because data.context is null.
It comes from the fact that when you touch the screen with your second finger the press method is deleting data.context but _not_ clearing the timeout that will call _isTap later on and access the data.context.
Attached in a small patch for this issue.
(this is reproducible on mobileCharting demo)
Attachments (1)
Change History (5)
comment:1 Changed 9 years ago by
| Component: | General → Dojox |
|---|---|
| Milestone: | tbd → 1.8 |
| Owner: | set to Adam Peller |
comment:2 Changed 9 years ago by
| Owner: | changed from Adam Peller to Evan |
|---|---|
| Status: | new → assigned |
Changed 9 years ago by
| Attachment: | 15188.patch added |
|---|
comment:3 Changed 9 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
comment:4 Changed 9 years ago by
| Version: | → 1.7.0 |
|---|
Note: See
TracTickets for help on using
tickets.
In [28802]:
Also back ported to 1.7.x branch