Closed
Bug 156454
Opened 23 years ago
Closed 23 years ago
DOM Mouse Scroll events should be handled by a <handler>
Categories
(Core :: XUL, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: neil, Assigned: hewitt)
References
Details
(Keywords: polish)
Attachments
(1 file, 1 obsolete file)
|
3.78 KB,
patch
|
janv
:
review+
bzbarsky
:
superreview+
|
Details | Diff | Splinter Review |
Currently a DOM Mouse Scroll event listener is added (and never removed!) to a
listbox or tree widget in its XBL constructor but the code would be simpler if
the event listener was written as an XBL handler.
|
Reporter | |
Comment 1•23 years ago
|
||
|
|
Reporter | |
Updated•23 years ago
|
|
||
Comment 2•23 years ago
|
||
*** Bug 156453 has been marked as a duplicate of this bug. ***
|
||
Comment 3•23 years ago
|
||
Comment on attachment 90608 [details] [diff] [review]
Proposed patch
r=varga
Attachment #90608 -
Flags: review+
|
|
Reporter | |
Comment 4•23 years ago
|
||
Pike spotted an aEvent.detail which should have been converted to rows.
Attachment #90608 -
Attachment is obsolete: true
|
|
||
Comment 5•23 years ago
|
||
Comment on attachment 91631 [details] [diff] [review]
Fixed patch
r=varga
Attachment #91631 -
Flags: review+
|
|
Reporter | |
Updated•23 years ago
|
Attachment #91631 -
Flags: superreview?(bzbarsky)
|
||
Updated•23 years ago
|
Attachment #91631 -
Flags: superreview?(bzbarsky) → superreview+
|
|
Reporter | |
Comment 6•23 years ago
|
||
Fix was checked in by timeless.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: shrir → xptoolkit.widgets
You need to log in
before you can comment on or make changes to this bug.
Description
•