Bug 45940 – REGRESSION (r67166): "Placeholder" text remains in input box after 2nd focus()

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Joel]

http://le.bytestudios.com/1.php

focus and blur multiple times to see it happen

noticed this very recently in the latest nightly, safari 5 works as expected.

[Paul Irish]

Demo page: http://www.thecssninja.com/demo/placeholder_issue/

chromium ticket: http://code.google.com/p/chromium/issues/detail?id=56226

repro in latest chromium build and webkit nightly.

[Andy Estes]

Created attachment 68745 [details]
Reduction

[Joel]

This might break some w3c guideline. but wouldn't it be nice to dim the placeholder text, instead of completely remove it on focus? that way you still know what you need to enter when you tab. keypress would clear the placeholder completely

[Andy Estes]

This regressed somewhere between r67129 and r67358.

[Andy Estes]

Looks like this regressed in <http://trac.webkit.org/changeset/67166>.

[Andy Estes]

(In reply to comment #3)
> This might break some w3c guideline. but wouldn't it be nice to dim the placeholder text, instead of completely remove it on focus? that way you still know what you need to enter when you tab. keypress would clear the placeholder completely

From <http://dev.w3.org/html5/spec/Overview.html#attr-input-placeholder>:

"User agents should present this hint to the user, after having stripped line breaks from it, when the element's value is the empty string and the control is not focused (e.g. by displaying it inside a blank unfocused control)."

Clearing the placeholder text on focus sounds like the right thing to do based on the spec. This is off the topic of this bug, but you could probably accomplish what you describe by listening to focus and keydown events in JavaScript.

[Kent Tamura]

Created attachment 68878 [details]
Patch

[Kyle Fox]

I just noticed this behaviour with an <input> with autofocus and filed a bug (https://bugs.webkit.org/show_bug.cgi?id=46757) before finding this ticket.

It's pretty much the same issue as this, however.

[Kent Tamura]

*** Bug 46757 has been marked as a duplicate of this bug. ***

<rdar://problem/8497335>

Created attachment 69469 [details]
Alternate fix

The attached patch is an alternate solution. Since the check before setting the text value in the HTMLInputElement is to ensure that the value is 'acceptable', the patch checks for the hasUnacceptableValue() instead of formControlValueMatchesRenderer().

#16465 may also be this issue.

[Kent Tamura]

(In reply to comment #11)
> Created an attachment (id=69469) [details]
> Alternate fix
> 
> The attached patch is an alternate solution. Since the check before setting the text value in the HTMLInputElement is to ensure that the value is 'acceptable', the patch checks for the hasUnacceptableValue() instead of formControlValueMatchesRenderer().

Your patch looks to have good readability.  Would you add ChangeLog and copy the test from my patch please?

> #16465 may also be this issue.

I don't think so.

[Sadrul Habib Chowdhury]

Created attachment 69675 [details]
Updated patch

Added the layouttests and changelogs from Kent's patch.

[WebKit Commit Bot]

Comment on attachment 69675 [details]
Updated patch

Rejecting patch 69675 from commit-queue.

Failed to run "['./WebKitTools/Scripts/webkit-patch', '--status-host=queues.webkit.org', 'apply-attachment', '--force-clean', '--non-interactive', '--quiet', 69675]" exit_code: 2
Cleaning working directory
Updating working directory
Logging in as [e-mail]...
Fetching: https://bugs.webkit.org/attachment.cgi?id=69675&action=edit
Fetching: https://bugs.webkit.org/show_bug.cgi?id=45940&ctype=xml
Processing 1 patch from 1 bug.
Processing patch 69675 from bug 45940.
Failed to run "[u'/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Darin Adler', u'--force']" exit_code: 9

Full output: http://queues.webkit.org/results/4172080

[Kent Tamura]

Comment on attachment 69675 [details]
Updated patch

I tried sadrul's patch locally and found it broke another test.
My patch should be necessary and sufficient.

[Dimitri Glazkov (Google)]

Comment on attachment 69675 [details]
Updated patch

ok.

[Dimitri Glazkov (Google)]

Comment on attachment 69675 [details]
Updated patch

doh.

[Dimitri Glazkov (Google)]

Comment on attachment 68878 [details]
Patch

ok.

[Kent Tamura]

(In reply to comment #7)
> Created an attachment (id=68878) [details]
> Patch

Committed this one as r69176.