Skip to content
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

[css.types.url] Add src() function #25081

Closed
wants to merge 2 commits into from

Conversation

ExE-Boss
Copy link
Contributor

Summary

This adds initial compatibility data for the new CSS Values 4 src(…) function, which is like the url(…) function, except it doesn’t support the unquoted legacy syntax, so that src(var(‑‑foo)) is valid.

Test results and supporting details

@github-actions github-actions bot added the data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS label Nov 15, 2024
Copy link
Contributor

@caugner caugner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your PR.

However, BCD's data guidelines state:

Browser features that have not been implemented (or planned to be implemented) in any browser, should not be added to BCD.

We cannot consider the Firefox bug you referenced (and created yourself in 2021) as evidence for intent to implement, as the bug is unassigned and only has severity S3.

So I'm going ahead and close this PR.

Please feel free to reopen this when any browser shows intent to implement.

@caugner caugner closed this Nov 15, 2024
@ExE-Boss
Copy link
Contributor Author

ExE-Boss commented Nov 16, 2024

The fact that this isn’t listed anywhere else as unimplemented tripped me up when I tried to use src(…) in my newer stylesheets. (took a while to debug as @import fails aren’t logged anywhere in the dev tools)

It should probably be marked as such in bcd‑content for Web/CSS/<url> § src().

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants