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

Mismatch bcd key for url() function #25413

Open
skyclouds2001 opened this issue Dec 14, 2024 · 7 comments
Open

Mismatch bcd key for url() function #25413

skyclouds2001 opened this issue Dec 14, 2024 · 7 comments
Assignees
Labels
data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS

Comments

@skyclouds2001
Copy link
Contributor

MDN URL

https://developer.mozilla.org/en-US/docs/Web/CSS/url_function

What specific section or headline is this issue about?

Browser compatibility

What information was incorrect, unhelpful, or incomplete?

the bcd key refers to the <url> css type, not the url() function

What did you expect to see?

fix the bcd key

Do you have any supporting links, references, or citations?

No response

Do you have anything more you want to share?

No response

MDN metadata

Page report details
@skyclouds2001 skyclouds2001 added the needs triage This issue needs to be confirmed label Dec 14, 2024
@hamishwillee
Copy link
Contributor

@skyclouds2001 This is confusing, but I think it is correct. The <url> value (data type) takes a value that is the url() function - they are indivisible. So they have the same data.

@estelle Is the documentation of the <url> type in any way useful - I mean does this actually every appear in "code" or do you only ever see url()? If not, it might make sense to not document it and point to the URL function everywhere.

@skyclouds2001
Copy link
Contributor Author

The <url> value (data type) takes a value that is the url() function - they are indivisible. So they have the same data.

currently it is true, as src() didn't support by any browsers, so <url> = <url()>; but the spec defines: <url> = <url()> | <src()>, so I think we should make <url> and url() have different BCD items

see also mdn/content#36839

Is the documentation of the <url> type in any way useful - I mean does this actually every appear in "code" or do you only ever see url()? If not, it might make sense to not document it and point to the URL function everywhere.

Personally I think it is better to still keep them as two pages at present

@hamishwillee
Copy link
Contributor

hamishwillee commented Dec 16, 2024

@skyclouds2001 Perhaps! I am not sure because even if <url>=<src()> I still see not much value in <url> except as a syntax glue for the spec. We might not have value for it in the docs.

But anyhow, the question here is really about BCD in the first case. There is no url() BCD entry so we can't do anything with the docs unless the BCD team agree that it makes sense to do so. Transferring the issue.

BCD compat team, do we need a separate url() feature given the argument above. My feeling is perhaps this would be a subfeature of <url>, but I really don't know.

@hamishwillee hamishwillee transferred this issue from mdn/content Dec 16, 2024
@queengooborg queengooborg added data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS and removed needs triage This issue needs to be confirmed labels Dec 16, 2024
@estelle
Copy link
Member

estelle commented Dec 17, 2024

@estelle Is the documentation of the <url> type in any way useful - I mean does this actually every appear in "code" or do you only ever see url()? If not, it might make sense to not document it and point to the URL function everywhere.

What @skyclouds2001 said.
That page as written is useless, but the page is needed. we could do a much better job
For example "The value is either of the following:" and then there is only one value?!?!
We do document (or try to document) all the data types, and <url> is a data type, so must be documented. It is found in multiple property values, so needs its own page (there are some data types that are only on the property page for which they are a value)

@hamishwillee
Copy link
Contributor

Great. So I guess the first thing is (still) to get a feature for the url() function and look at the docs for this as a separate problem.

@skyclouds2001
Copy link
Contributor Author

skyclouds2001 commented Dec 17, 2024

For example "The value is either of the following:" and then there is only one value?!?!

I guess this is because previously there is also a src() function, but since it is not supported by any browsers at present, it was removed via mdn/content#36839.

See also the note just below that paragraph.

@skyclouds2001
Copy link
Contributor Author

Great. So I guess the first thing is (still) to get a feature for the function and look at the docs for this as a separate problem.url()

yes, and I guess the css url() function should have the same bcd info as the css <url> type

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

No branches or pull requests

4 participants