RESOLVED WONTFIX 11800
REGRESSION from Safari 2.0.4 (419.3): Sunken text in YouGov login page box 
https://bugs.webkit.org/show_bug.cgi?id=11800
Summary REGRESSION from Safari 2.0.4 (419.3): Sunken text in YouGov login page box
Stephen Harbage
Reported 2006-12-10 16:44:28 PST
If you go to the YouGov website login and type into the user name and password box you can see that the text is sunken. Works fine on the released version 2.0.4 (419.3) of Safari on 10.4.8
Attachments
Example of sunken text (2.06 KB, image/png)
2006-12-10 16:46 PST, Stephen Harbage 		no flags
Details
Example of sunken text (2.06 KB, image/png)
2006-12-10 16:46 PST, Stephen Harbage 		no flags
Details
Rendering in Firefox 2.0 (4.05 KB, image/png)
2006-12-10 22:27 PST, mitz 		no flags
Details
Rendering in IE7 Windows XP (31.18 KB, image/png)
2007-02-08 00:01 PST, Charles Ying 		no flags
Details
Minimal test case (375 bytes, text/html)
2007-02-18 00:16 PST, Oliver Hunt 		no flags
Details
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Stephen Harbage
Comment 1 2006-12-10 16:46:27 PST
Created attachment 11796 [details] Example of sunken text
Stephen Harbage
Comment 2 2006-12-10 16:46:27 PST
Created attachment 11797 [details] Example of sunken text
David Kilzer (:ddkilzer)
Comment 3 2006-12-10 17:02:17 PST
Confirmed on locally-built debug build of Webkit r18138 on Mac OS X 10.4.8 (8L127).
mitz
Comment 4 2006-12-10 22:27:59 PST
Created attachment 11798 [details] Rendering in Firefox 2.0 It doesn't look any better in Firefox 2.0
David Kilzer (:ddkilzer)
Comment 5 2006-12-11 03:35:15 PST
(In reply to comment #4) > It doesn't look any better in Firefox 2.0 Opera 9.02 and OmniWeb 5.5 look okay, though.
Maciej Stachowiak
Comment 6 2007-01-29 03:49:05 PST
<rdar://problem/4960646>
Maciej Stachowiak
Comment 7 2007-02-07 05:35:43 PST
I think the problem is the padding set on the input elements, which perhaps Opera and OmniWeb don't respect. Probably an evangelism issue.
Charles Ying
Comment 8 2007-02-08 00:01:58 PST
Created attachment 13048 [details] Rendering in IE7 Windows XP
Oliver Hunt
Comment 9 2007-02-18 00:16:40 PST
Created attachment 13220 [details] Minimal test case Minimal test case, from this it looks like they have set the height of the text field to be shorter than the font height. This test case is minimised, and the style is extracted from their style sheet.. though obviously not the amroon bit, i wanted some contrast :D
Dave Hyatt
Comment 10 2007-02-18 00:35:50 PST
We should consider enforcing a min-height on text fields by default that is the border + padding + line of text, i.e., enforce a minimum height = to the intrinsic height.
Oliver Hunt
Comment 11 2007-02-18 00:45:50 PST
This isn't worth fixing -- the page doesn't work in IE or FFX and we are rendering what we should be. It isn't the place of the browser to work out whether the author has done something silly
Note You need to log in before you can comment on or make changes to this bug.