see attached testcase. actual: 1. if the size attribute is not defined, but multiple attribute is defined - the select is displayed as a drop down box. 2. user unable to select multiple options expected: 1. the select should be displayed as a list box with visible options equal to the size given. 2. user should be able to select multiple options using mouse or the keyboard.
Assignee: rods → bryner
Created attachment 91917 [details] [diff] [review] Patch v.1.0 Includes indentation changes since some things got accidentally screwed up. See the previous attachment for a diff -w of this patch. Also fixes a CSS parser error.
I played with keeping the current implementation, but this makes it alot easier, plus in some ways, the way I have this makes much more sense. Changes: frame controls didn't expose a .size on select multiple with no size attribute. We now do that per the spec. This made it much more difficult to do our previous frame control implementation where we defaulted size to however many options were under the select, and then resized the select on adding or removing elements from it with no size attribute. As a result, I now have a default value. MSIE uses 4 as their default value so I decided to do the same. I also disabled setting the height since height seems to me to be pretty useless: The spec says that the size attribute is what specifies how many rows should be displayed: http://www.w3.org/TR/html401/interact/forms.html#adef-size-SELECT and the height style we know to affect the size of the box itself. So setting height will lead to weird combinations with the select, e.g. <select size=3 style="height: 500px"> will lead to a funky select with 3 items visible, but a lot of whitespace.... I wouldn't mind leaving that on, but I somehow think that we want to prevent users from doing the stupid thing here.
Assignee: bryner → caillon
Assignee: caillon → form
QA Contact: tpreston → desale
Assignee: layout.form-controls → nobody
QA Contact: desale → layout.form-controls
XBL form control bugs are no longer *especially* relevant to anyone.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.