Closed
Bug 310647
Opened 19 years ago
Closed 3 years ago
HTML Select change event fires when it shouldn't
Categories
(Core :: Layout: Form Controls, defect)
Tracking
()
RESOLVED
INCOMPLETE
People
(Reporter: mikecaines, Unassigned, NeedInfo)
References
Details
(Keywords: testcase)
Attachments
(1 file)
(deleted),
text/html
|
Details |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b5) Gecko/20050926 Firefox/1.4
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b5) Gecko/20050926 Firefox/1.4
The change event is firing when changing and undoing a selection of a
non-drop-down select element, within the same mouse drag action.
Reproducible: Always
Steps to Reproduce:
1. Mousedown on the first option of the first select
2. Move the cursor down over the second option
3. Move the cursor back up over the first option
4. Mouseup over the first option
Actual Results:
Change event is fired
Expected Results:
Change event is not fired
The source of this may be here:
http://lxr.mozilla.org/mozilla1.8/source/layout/forms/nsListControlFrame.cpp#2557
as mChangesSinceDragStart gets set to PR_TRUE during the mouse drag action, but
with this particular select situation a change event should only be dispatched
if the selected options on mouseup are different from the selected options on
mousedown.
Comment 1•19 years ago
|
||
Could you give a testcase? I failed to make one that shows the problem.
Reporter | ||
Comment 2•19 years ago
|
||
Comment 3•19 years ago
|
||
Thanks! Stupid of me, I was trying a drop-down select, while you specifically
said it was a non-drop-down select that has the bug.
This bug is also noticable in Mozilla1.7
The lxr link you gave points to changes made by Robert in bug 207637. CC-ing him.
Assignee: events → nobody
Status: UNCONFIRMED → NEW
Component: DOM: Events → Layout: Form Controls
Ever confirmed: true
Keywords: testcase
QA Contact: ian → layout.form-controls
Flags: blocking1.9a1? → blocking1.9-
Comment 4•3 years ago
|
||
Hey Mike,
Can you still reproduce this issue or should we close it?
Flags: needinfo?(mikecaines)
Comment 5•3 years ago
|
||
Marking this as Resolved > Incomplete since the last activity on this issue was 15 years ago and it might not be relevant anymore.
Feel free to re-open if the issue is still reproducible on your end in the latest FF versions.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → INCOMPLETE
You need to log in
before you can comment on or make changes to this bug.
Description
•