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_ref broken links #132

Open
estelle opened this issue Feb 13, 2025 · 1 comment
Open

css_ref broken links #132

estelle opened this issue Feb 13, 2025 · 1 comment
Labels
needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened.

Comments

@estelle
Copy link
Member

estelle commented Feb 13, 2025

What information was incorrect, unhelpful, or incomplete?

https://developer.mozilla.org/en-US/docs/Web/CSS/Reference#index, created with a macro call to {{css_ref}} lists pages as missing even though they are documented.

color-mix(), ::cue, :has, :is, layer, :where(), and many other pages in red do exist

some are in subdirectories (https://developer.mozilla.org/en-US/docs/Web/CSS/color_value/color-mix, https://developer.mozilla.org/en-US/docs/Web/CSS/@import/layer_function)
others have parenthesis that aren't in the slug (https://developer.mozilla.org/en-US/docs/Web/CSS/:has, https://developer.mozilla.org/en-US/docs/Web/CSS/:is, https://developer.mozilla.org/en-US/docs/Web/CSS/:where)

What did you expect to see?

blue links

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

see above comments

Do you have anything more you want to share?

https://github.com/mdn/rari/blob/main/crates/rari-doc/src/templ/templs/css_ref.rs

@estelle estelle added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Feb 13, 2025
@caugner
Copy link
Contributor

caugner commented Feb 24, 2025

@fiji-flo Could this be fixed by adding css/functions.json to MDNDataFiles, and using the mdn_url from there? I guess the current color-mix() entry comes from css/syntaxes.json, which doesn't have mdn_url context.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened.
Projects
None yet
Development

No branches or pull requests

2 participants