Closed Bug 1470025 Opened 7 years ago Closed 4 months ago

User Agent stylesheet default opacity for placeholder text

Categories

(Core :: Layout: Form Controls, enhancement)

62 Branch
enhancement

Tracking

()

RESOLVED FIXED
132 Branch
Webcompat Priority P2
Tracking Status
firefox62 --- wontfix
firefox132 --- fixed

People

(Reporter: karlcow, Assigned: emilio)

References

Details

(Whiteboard: [webcompat:p2][needs-wpt-?])

User Story

Would be good for GeckoView.

Attachments

(2 files)

This will be used to track the webcompat issues related to the differences of default opacity in between Gecko and WebKit/Blink. Firefox: 0.54 WebKit/Blink: 1 * The CSS WG agreed that User Agent Stylesheets should be tested. https://github.com/w3c/web-platform-tests/issues/5625 * Someone at Google thinks it would be good to define what it should for interop https://github.com/whatwg/html/issues/2561#issuecomment-296130295
Flags: webcompat?
Would be good to fix this issue interop for GeckoView.
Whiteboard: [webcompat] → [webcompat:p3]
User Story: (updated)
Whiteboard: [webcompat:p3] → [webcompat:p2]
Flags: webcompat? → webcompat+

See bug 1547409. Migrating webcompat priority whiteboard tags to project flags.

Webcompat Priority: --- → P2

There does not seem to have been any progress on this despite the WG agreeing that it should be done. As such we still need web platform tests here.

Whiteboard: [webcompat:p2] → [webcompat:p2][needs-wpt-?]
See Also: → 1585764
Attached file GitHub Pull Request
Assignee: nobody → kdubost

In Gecko:
Perhaps use ::placeholder { opacity: 1.0; color: color-mix(in srgb, currentColor 54%, transparent); }
or use ::placeholder { opacity: 1.0; color: rgb(from currentColor, r g b / 54%); }
once css-color-5 Relative RGB colors are supported.
And possibly alias ::-webkit-input-placeholder to ::placeholder

In Blink:
Create dark color-scheme
::-webkit-input-placeholder { color: -internal-light-dark() }
and use that until one of the two methods above.

Sorry, there was a problem with the detection of inactive users. I'm reverting the change.

Assignee: nobody → karl+moz

Actually, Karl isn't with Mozilla anymore, so we might we as well unassign the bug for now.

Assignee: karl+moz → nobody
Severity: normal → S3
Duplicate of this bug: 1884709

Doing something like the color-mix in comment 6 seems reasonable to me, fwiw.

This keeps our rendering, but using the same property as other browsers,
which prevents these issues.

Assignee: nobody → emilio
Status: NEW → ASSIGNED
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/aae10b80691a Use color rather than opacity for placeholder. r=dholbert
Pushed by emilio@crisal.io: https://hg.mozilla.org/integration/autoland/rev/fc77cd3a4c25 Annotate one passing reftest on Windows.
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → 132 Branch
Regressions: 1921995
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: