[Webkit-unassigned] [Bug 63927] [META] REGRESSION(r72052): Multiple placeholder regressions

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jul 5 20:19:02 PDT 2011


--- Comment #1 from Kent Tamura <tkent at chromium.org>  2011-07-05 20:19:02 PST ---
I tried to represent a placeholder text as a shadow node, and replace the inner editable node with the placeholder node.  The code change for this approach was really simple.
However I had no good idea for implementing Bug 53740 behavior.
 - How to show the text caret with the placeholder text?
 - How to handle context menu on the placeholder node?

I tried to implement another approach. Representing a placeholder text as a shadow node, it has display:none if the control has non-empty value, and show the placeholder node if the placeholder should be visible.
This approach requires to add a lot of custom layout code in RenderTextControlSingleLine.cpp. I don't want to make it more complex ;-(

Fixing text-align and line-height issues in the current implementation is not hard. I guess fixing the wrapping issue, Bug 51290, would be hard and boring.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list