-
Notifications
You must be signed in to change notification settings - Fork 715
[css-scoping-1] ::placeholder selector inside ::slotted() not working #3150
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
|
Yeah, the argument to But yeah, ( |
Actually I think this is a valid issue. The resolution in #1747 only referenced tree-abiding pseudos (::before / ::after). I actually have an implementation that works for all pseudos, but I don't plant to land it without a resolution here. |
See w3c/csswg-drafts#3150 for the issue that would expand this to all pseudos. Differential Revision: https://phabricator.services.mozilla.com/D9994
See w3c/csswg-drafts#3150 for the issue that would expand this to all pseudos. Differential Revision: https://phabricator.services.mozilla.com/D9994 bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1502617 gecko-commit: b7ed6370bc7d11c486ca86f74ba7e4bc00b471af gecko-integration-branch: mozilla-inbound gecko-reviewers: heycam
See w3c/csswg-drafts#3150 for the issue that would expand this to all pseudos. Differential Revision: https://phabricator.services.mozilla.com/D9994
See w3c/csswg-drafts#3150 for the issue that would expand this to all pseudos. Differential Revision: https://phabricator.services.mozilla.com/D9994 bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1502617 gecko-commit: b7ed6370bc7d11c486ca86f74ba7e4bc00b471af gecko-integration-branch: mozilla-inbound gecko-reviewers: heycam
See w3c/csswg-drafts#3150 for the issue that would expand this to all pseudos. Differential Revision: https://phabricator.services.mozilla.com/D9994
See w3c/csswg-drafts#3150 for the issue that would expand this to all pseudos. Differential Revision: https://phabricator.services.mozilla.com/D9994
According to css-pseudo-4, the tree-abiding pseudos are: ::before, ::after, ::marker, ::placeholder. So it currently follows from the specs that ::placeholder should be allowed after ::slotted(). There's no limitation in Blink that makes it hard to support any kind of pseudo elements after ::slotted(), but the previous CSSWG resolution was to limit this. Reported Blink issue: https://crbug.com/902518 |
Ah, alright, true that. I'll fixup firefox as well. |
(I agree this is invalid then, just a browser bug) |
Wasn't quite as simple in Blink: https://chromium-review.googlesource.com/c/chromium/src/+/1323049 |
See w3c/csswg-drafts#3150 for the issue that would expand this to all pseudos. Differential Revision: https://phabricator.services.mozilla.com/D9994 UltraBlame original commit: b7ed6370bc7d11c486ca86f74ba7e4bc00b471af
See w3c/csswg-drafts#3150 for the issue that would expand this to all pseudos. Differential Revision: https://phabricator.services.mozilla.com/D9994 UltraBlame original commit: b7ed6370bc7d11c486ca86f74ba7e4bc00b471af
See w3c/csswg-drafts#3150 for the issue that would expand this to all pseudos. Differential Revision: https://phabricator.services.mozilla.com/D9994 UltraBlame original commit: b7ed6370bc7d11c486ca86f74ba7e4bc00b471af
Adding here in case others are searching for this: Webkit has yet to fix the bug https://bugs.webkit.org/show_bug.cgi?id=223814 |
Referencing this issue #1747
Working as intended:
::slotted(::hover){ color: red; }
Not working:
::slotted(::placeholder){ color: red; }
From the issue I am referencing, it seems like it is supposed to not work in both scenarios, as option 3 was chosen to be allowed and option 2 was not. So basically, ::slotted()::hover should work but ::slotted(::hover) should not. At least, this was my interpretation of that issue.
Could someone give some clarification on what should work and what should not, and perhaps why in my case the ::placeholder selector within ::slotted does not work?
The text was updated successfully, but these errors were encountered: