[go: up one dir, main page]

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

Should other attributes be linked for consistency in the formal syntax? #11528

Open
3 tasks done
mirunacurtean opened this issue Jul 23, 2024 · 1 comment
Open
3 tasks done
Labels
on hold Waiting on something else before this can be moved forward.

Comments

@mirunacurtean
Copy link

Summary

example on https://developer.mozilla.org/en-US/docs/Web/CSS/calc#formal_syntax with https://developer.mozilla.org/en-US/docs/Web/CSS/calc-keyword
→ ❓Probably makes sense to link somehow, but needs discussion, yet is low priority. Wait for the rewrite to be done, then bring it up in 2025.
From @LeoMcA : I believe this is because the definition is expanded in calc(), so there's no need to link to it to explain it - whether it should still be linked for consistency with the other keywords is another question

URL

https://developer.mozilla.org/en-US/docs/Web/CSS/calc#formal_syntax

Reproduction steps

  1. Copy the link in the browser URL bar https://developer.mozilla.org/en-US/docs/Web/CSS/calc#formal_syntax
  2. Observe the linked attributes.
    3.Observe that not all attributes are links, despite existing reference pages.

Expected behavior

Discuss and establish whether all attributes should be linked for consistency.

Actual behavior

Inconsistent behavior.
image

Device

Desktop

Browser

Chrome

Browser version

Stable

Operating system

Android

Screenshot

image

Anything else?

No response

Validations

@github-actions github-actions bot added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Jul 23, 2024
@Josh-Cena
Copy link
Member

Hello! calc-keyword was only recently renamed: mdn/content#34348 I believe it was linking previously, but I'm not sure how these links are coded. Maybe it also needs a Yari update.

@argl argl added on hold Waiting on something else before this can be moved forward. and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
on hold Waiting on something else before this can be moved forward.
Projects
None yet
Development

No branches or pull requests

3 participants