Bug 11800 - REGRESSION from Safari 2.0.4 (419.3): Sunken text in YouGov login page box
Summary: REGRESSION from Safari 2.0.4 (419.3): Sunken text in YouGov login page box
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: Evangelism (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P1 Normal
Assignee: Nobody
URL: http://www.yougov.com/Users/Login.asp
Keywords: InRadar, NeedsReduction, Regression
Depends on:
Blocks:
 
Reported: 2006-12-10 16:44 PST by Stephen Harbage
Modified: 2007-02-18 00:45 PST (History)
3 users (show)

See Also:

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 an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Stephen Harbage 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
Comment 1 Stephen Harbage 2006-12-10 16:46:27 PST 
Created attachment 11796 [details]
Example of sunken text
Comment 2 Stephen Harbage 2006-12-10 16:46:27 PST 
Created attachment 11797 [details]
Example of sunken text
Comment 3 David Kilzer (:ddkilzer) 2006-12-10 17:02:17 PST 
Confirmed on locally-built debug build of Webkit r18138 on Mac OS X 10.4.8 (8L127).
Comment 4 mitz 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
Comment 5 David Kilzer (:ddkilzer) 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.
Comment 6 Maciej Stachowiak 2007-01-29 03:49:05 PST 
<rdar://problem/4960646>
Comment 7 Maciej Stachowiak 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.
Comment 8 Charles Ying 2007-02-08 00:01:58 PST 
Created attachment 13048 [details]
Rendering in IE7 Windows XP
Comment 9 Oliver Hunt 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
Comment 10 Dave Hyatt 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.
Comment 11 Oliver Hunt 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