-
Notifications
You must be signed in to change notification settings - Fork 707
[css-content] content: contents will not be compatible. #4632
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
Changing the computed value of If #foo { content: normal }
#foo::before { content: inherit } the I'm also not sure about computing #foo::before {
content: normal;
display: list-item;
}
#foo::before::marker {
content: inherit;
} the So I support |
We got the The distinction between Here's a slightly contrived example of how the <style>
a.external {
content: none;
}
a.external::before {
content: "(" contents ")";
}
a.external::after {
content: url(https://upload.wikimedia.org/wikipedia/commons/2/25/External.svg);
height: 1em;
}
</style>
See <a class="external" href="http://w3.org">w3.org</a> for detail -- I also think removing |
The problem here is |
Yes, that's fair. I was missing the subtle distinction there. But in any case |
I was fixing a Gecko compat issue (webcompat/web-bugs#46914) which is present because we always serialize
content: normal
ascontent: none
, even on elements.I'm going to fix Gecko to follow the spec and
getComputedStyle()
returnsnone
for pseudo-elements, but not for elements. But such code wouldn't work if we were to makecontent: normal
compute tocontent: contents
on elements either, which is what the latest draft specifies: https://drafts.csswg.org/css-content/#content-propertySo I propose for
content: normal
to compute to itself for elements, and removingcontent: contents
from the spec.The text was updated successfully, but these errors were encountered: