RESOLVED INVALID 10137
REGRESSION: Width style attribute not applied the same to unstyled <input type="text"> in WebKit as for Safari
https://bugs.webkit.org/show_bug.cgi?id=10137
Summary REGRESSION: Width style attribute not applied the same to unstyled <input typ...
Jason Prell (superrcat)
Reported 2006-07-27 20:15:40 PDT
Using 2006-07-28 r15664 of WebKit on Mac OS X v10.4.7 PPC, input elements with a type of password are not the correct width when the width is set with the style attribute. The elements below will render with different widths using r15664: <input name="txtUserName" type="text" id="txtUserName" tabindex="1" style="width:130px;" /> <input name="txtPassword" type="password" id="txtPassword" tabindex="2" style="width:130px;" /> This is a regression and did not occur with WebKit/418.8 Safari/419.3 (2.0.4). An example can be viewed at the login page for the URL on this bug. Attached are also images from r15664 and WebKit/418.8.
Attachments
input text and password type rendering with styled width on 418.8 (9.03 KB, image/png)
2006-07-27 20:16 PDT, Jason Prell (superrcat)
no flags
input text and password type rendering with styled width on r15664 (9.03 KB, image/png)
2006-07-27 20:18 PDT, Jason Prell (superrcat)
no flags
Reduction for this bug (475 bytes, text/html)
2006-08-08 22:34 PDT, Jason Prell (superrcat)
no flags
Jason Prell (superrcat)
Comment 1 2006-07-27 20:16:57 PDT
Created attachment 9729 [details] input text and password type rendering with styled width on 418.8
Jason Prell (superrcat)
Comment 2 2006-07-27 20:18:02 PDT
Created attachment 9730 [details] input text and password type rendering with styled width on r15664
Jason Prell (superrcat)
Comment 3 2006-07-27 20:27:25 PDT
The problem with password fields is tracked in <a href="show_bug.cgi?id=6990" title="ASSIGNED - Switch to use new text field implementation for &lt;input type=&quot;password&quot;&gt;">bug 6990</a>. Adding dependency.
Gustaaf Groenendaal (MysteryQuest)
Comment 4 2006-07-28 04:51:28 PDT
Confirmed. The bug does not seem to be caused by incorrect handling of the width element, but comes from somewhere else, probably a NativeTextfield issue.
David Kilzer (:ddkilzer)
Comment 5 2006-07-28 05:49:13 PDT
Regressions are P1. Adding NeedsRadar keyword.
jonathanjohnsson
Comment 6 2006-08-05 07:44:59 PDT
The password field actually has the width specified, 130 px, when I measure it. It's the normal text field that has the wrong length, 136 px.
jonathanjohnsson
Comment 7 2006-08-05 08:14:52 PDT
I tried with widths down to 0px, and the normal text field always has a real ("outer") width that is 6px more than the declared, while the password text field stays at the exact declared value. For example, a text field of width 0px is actually 6px wide, while a password text field with width 0px really is 0px wide, i.e. invisible. For comparison, Firefox 1.5 adds 4px for each field ("outer" width), compared to declared width. The normal text field and the password text field should at least be the same "outer" width, for same declared. Changing summary. I measured with "Free Ruler".
jonathanjohnsson
Comment 8 2006-08-05 08:55:29 PDT
Learning more about how width styles and such work for text inputs, it seems parts of my last comment are irrelevant, including, probably the new summary. :( In any case, it is the text field visible width that has changed from Safari to latest WebKit, for the same applied style width. I think this is the real regression, so changing summary again.
Jason Prell (superrcat)
Comment 9 2006-08-08 18:05:41 PDT
Jason Prell (superrcat)
Comment 10 2006-08-08 22:34:58 PDT
Created attachment 9952 [details] Reduction for this bug Attaching reduction for this bug.
Adele Peterson
Comment 11 2006-08-11 12:33:23 PDT
This is caused by a change to match Firefox & IE that makes text fields and textareas use the correct box model when in strict mode. Password fields haven't been transitioned to the new text field implementation yet, so that's why they're different. If you remove the doctype, you'll see that in quirks mode, our new text fields will begin including the border & padding in the width, and in the test case both fields will have matching widths. When the password fields are fully transitioned, you'll see that in strict mode, the specified width will *not* include border and padding for both kinds of text fields.
Note You need to log in before you can comment on or make changes to this bug.