Simplify placeholder / autofill preview visibility code.
Categories
(Core :: DOM: Core & HTML, enhancement, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox88 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
Details
Attachments
(1 file)
I wrote this for bug 1698043 but it ended up not being how I fixed the bug, so splitting it up a bit.
Assignee | ||
Comment 1•3 years ago
|
||
Should be much simpler and doesn't need to deal with the different
stuff. We already have pseudo-classes for this, :autofill and
:placeholder-shown.
I initially wrote this because this is the only limitation that forces
us to have the placeholder text as a direct child of the text control
frame. In the end I kept that as-is, but this simplification is still
worth it.
We remove dom.placeholder.show_on_focus because it doesn't behave
correctly (it doesn't match the :placeholder-shown pseudo-class and it
should). It was introduced in bug 807613 and never turned to false by
default. I suspect nobody will miss this, but if somebody complains
about it we can reintroduce it properly (handling the pref in DOM
instead, changing the right state bits).
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d989f71f0a41 Manage placeholder and autofill preview visibility using CSS rather than custom code. r=masayuki
Updated•3 years ago
|
Comment 3•3 years ago
|
||
bugherder |
Description
•