97347 - Buttons do not obey width/height attrs

Status: RESOLVED WONTFIX

(Core :: Layout: Form Controls, defect, P2)

Description

[av (gone)]

I am not sure about the component, could be Form Submission.

This site is a Russian electronic book/music store.
1. Go to this site
2. Locate search engine interace on the lefthand side, it says NETSCAPE USERS
CLICK HERE and Russian Keyboard there. There is a nice feature for those who
don't have their machine russified: click on Russian Keyboard pops up a window
which resembles a keyboard and can be used to type chars in the search box.
3. Click on Russian Keyboard

Actual result: what we see with Mozilla (attachment follows)
Expected: what we see with 4x (attachment follows)

The direct link to 'keyboard' html code is http://www.st-p.com/Nkeyboard.asp

Comment 1

[av (gone)]

Attachment: The keyboard as displayed by 4.x

Comment 2

[av (gone)]

Attachment: The keyboard as displayed by Mozilla

Comment 3

[av (gone)]

Attachment: Localy saved html code in case the original link goes away

Comment 4

[Boris Zbarsky [:bzbarsky]]

looks like we ignore the illegal width and height attributes on <input
type="submit".  What we should consider doing is attribute mapping in quirks
mode only.  Using style="width: whatever; height: whatever" on that page works
beautifully, by the way.

Over to HTML form controls.

Comment 5

[rods (gone)]

Attachment: patch

Comment 6

[Peter Lubczynski]

Rod, shouldn't this use style to resolve width and height? What about the % case?

Comment 7

[Marc Attinasi]

Rod, does Nav only support PX width and height, or does it support others, like
percentage too? Also, we usually do this kind of thing in MapAttributesIntoRule
in the content element - I think that would be better.

Comment 8

[rods (gone)]

moving to 0.9.6

Comment 9

[Boris Zbarsky [:bzbarsky]]

Unless there is a _really_ good reason to fix this, I would suggest wontfix....
 If there is one and only one site using it, we should not make a quirk for it.

Comment 10

[Jon Hall]

Re: Comment 4, The button tag does still does not handle css height and width
correctly anyway. bug 128115

Comment 11

[Boris Zbarsky [:bzbarsky]]

I saw it.  I have it fixed in my tree, in fact.

Comment 12

[Boris Zbarsky [:bzbarsky]]

I just eliminated quirks reflow for buttons.  With my changes the keyboard looks
a lot more reasonable and I am somewhat tempted to wontfix this bug... Do people
feel that we should really support this in quirks?

Comment 13

[rods (gone)]

NO KILL quirks!

Comment 14

[Boris Zbarsky [:bzbarsky]]

Ok.  We don't support this.  We show the keyboard OK.  IE5 does not support it
either.  So the only thing supporting it is NS4....

Comment 15

[av (gone)]

...and IE6 :(