Ticket #4878 (closed bug: fixed)
val() tries to set multiple values on single select lists
| Reported by: | bennettmcelwee | Owned by: | |
|---|---|---|---|
| Priority: | low | Milestone: | 1.4 |
| Component: | core | Version: | 1.3.2 |
| Keywords: | select val | Cc: | |
| Blocking: | Blocked by: |
Description
jQuery.fn.val(value) will select every matching option in a select list, even if it's not a multiple select list. It's invalid HTML for more than one option to be selected. When it happens, the browser has to pick one to display. Firefox and Safari pick the first selected option; IE6 picks the last.
Correct behaviour should probably be to select just the first matching value in these cases.
I'm attaching a patch to attributes.js rev 6333 that fixes the problem, along with a simple test page that shows the problem on IE6.
Attachments
-
attributes.js-r6333-patch.txt
(417 bytes) -
added by bennettmcelwee 3 years ago.
- Patch for attributes.js rev 6333
-
val-singleselect-test.html
(1.0 KB) -
added by bennettmcelwee 3 years ago.
- Test file showing the problem
-
attributes.diff
(774 bytes) -
added by bennettmcelwee 3 years ago.
- Proper patch file for rev 6333.
Change History
Changed 3 years ago by bennettmcelwee
-
attachment
attributes.js-r6333-patch.txt
added
Changed 3 years ago by bennettmcelwee
-
attachment
val-singleselect-test.html
added
Test file showing the problem
Changed 3 years ago by bennettmcelwee
-
attachment
attributes.diff
added
Proper patch file for rev 6333.
comment:1 Changed 15 months ago by dmethvin
- Keywords needsreview added
- Status changed from new to open
Should we try to force an error case into correct behavior?
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
Patch for attributes.js rev 6333