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)

x86
Windows XP
defect
Not set
major

Tracking

()

RESOLVED INCOMPLETE

People

(Reporter: mikecaines, Unassigned, NeedInfo)

References

Details

(Keywords: testcase)

Attachments

(1 file)

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.
Could you give a testcase? I failed to make one that shows the problem.
Attached file Testcase (deleted) —
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
Blocks: 207637
Flags: blocking1.9a1?
Flags: blocking1.9a1? → blocking1.9-

Hey Mike,
Can you still reproduce this issue or should we close it?

Flags: needinfo?(mikecaines)

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.

Attachment

General

Creator:
Created:
Updated:
Size: