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

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=63927





--- Comment #1 from Kent Tamura <tkent at chromium.org>  2011-07-05 20:19:02 PST ---
A)
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?

B)
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 ;-(

C)
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.