132454 - [XBLFC] select with multiple attribute defined, size attribue not defined is diplayed as a drop down box

Status: RESOLVED INVALID

(Core :: Layout: Form Controls, defect)

Description

[Madhur Bhatia]

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.

Comment 1

[Madhur Bhatia]

Attachment: reduced testcase

Comment 2

[Boris Zbarsky [:bzbarsky]]

to rbryner

Comment 3

[Christopher Aillon (sabbatical, not receiving bugmail)]

Attachment: diff -w

Comment 4

[Christopher Aillon (sabbatical, not receiving bugmail)]

Attachment: 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.

Comment 5

[Christopher Aillon (sabbatical, not receiving bugmail)]

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.

Comment 6

[Christopher Aillon (sabbatical, not receiving bugmail)]

.

Comment 7

[Mike Shaver (:shaver -- probably not reading bugmail closely)]

XBL form control bugs are no longer *especially* relevant to anyone.