Closed Bug 471883 Opened 17 years ago Closed 17 years ago

Gesture events should be dispatched to the element under mouse, not to document.

Categories

(Core :: DOM: UI Events & Focus Handling, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: smaug, Assigned: smaug)

References

(Blocks 1 open bug)

Details

(Keywords: addon-compat, fixed1.9.1)

Attachments

(1 file)

No description provided.
Attached patch patchSplinter Review
I decided to not to change the DOM API of gestures events at this point. They are just dispatched to the element under mouse, not document under mouse. Most of the patch is just about changing the testcase.
Attachment #355151 - Flags: superreview?(roc)
Attachment #355151 - Flags: review?(roc)
Attachment #355151 - Flags: superreview?(roc)
Attachment #355151 - Flags: superreview+
Attachment #355151 - Flags: review?(roc)
Attachment #355151 - Flags: review+
I might want to get this to 1.9.1 too.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Attachment #355151 - Flags: approval1.9.1?
Blocks: 477863
What's the impact of changing this interface at this point?
That comment about interface is bogus now. The interface got changed in Bug 477863.
And the impact should be pretty small. Gesture events aren't exposed to content. And if some extensions use gestures, they may not even notice the change, because the change was mostly backwards compatible. (init***Event change wasn't backward compatible)
I asked the approval mainly because Bug 461500 is wanted+.
Comment on attachment 355151 [details] [diff] [review] patch a191=beltzner
Attachment #355151 - Flags: approval1.9.1? → approval1.9.1+
Keywords: fixed1.9.1
Component: Event Handling → User events and focus handling
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: