Bug 11923 - REGRESSION: Placeholder text in password field shows as bullets
Summary: REGRESSION: Placeholder text in password field shows as bullets
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Forms (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P1 Normal
Assignee: Antti Koivisto
URL: data:text/html,%3Cinput%20type=%22pas...
Keywords: HasReduction, InRadar, Regression
Depends on:
Blocks:
 
Reported: 2006-12-21 22:54 PST by Matt Lilek
Modified: 2007-02-14 03:05 PST (History)
0 users

See Also:

Attachments
disable text security temporary when showing placeholder (10.62 KB, patch)
2007-02-13 16:33 PST, Antti Koivisto 		hyatt: review+
Details | Formatted Diff | Diff
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 Matt Lilek 2006-12-21 22:54:37 PST 
Placeholder text on non-search field elements seems to have made a reappearance recently but when used on password fields, the placeholder is displayed as security bullets making it rather useless.
Comment 1 Mark Rowe (bdash) 2007-01-28 19:05:41 PST 
<rdar://problem/4960257>
Comment 2 Adele Peterson 2007-02-13 14:30:14 PST 
Right now, we have this rule in html4.css:

input[type="password"] {
    -webkit-text-security: disc !important;
}

We could remove this, and set the property in RenderTextControl.

We would need code in RenderTextControl::createInnerTextStyle and in RenderTextControl::updatePlaceholder.
Comment 3 Antti Koivisto 2007-02-13 16:33:50 PST 
Created attachment 13158 [details]
disable text security temporary when showing placeholder
Comment 4 Dave Hyatt 2007-02-14 02:04:07 PST 
Comment on attachment 13158 [details]
disable text security temporary when showing placeholder 

r=me

I wasn't aware we even supported placeholder text on non-search fields...
Comment 5 Antti Koivisto 2007-02-14 03:05:03 PST 
r19620