-
Notifications
You must be signed in to change notification settings - Fork 23k
chore(cssxref): Replace links with cssxref macro in web/api
#42434
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
Conversation
Search: \[`([^`]+?)`\]\(/en-US/docs/Web/CSS/Reference/Properties/\1\)
Replace: {{cssxref("$1")}}
Search: \[`([^`]+?)`\]\(/en-US/docs/Web/CSS/Reference/Selectors/\1\)
Replace: {{cssxref("$1")}}
Search: \[`([^`]+?)`\]\(/en-US/docs/Web/CSS/Reference/At-rules/\1\)
Replace: {{cssxref("$1")}}
Search: \[`([^/]+)\(\)`\]\(/en-US/docs/Web/CSS/Reference/Values/(filter-function)/\1\)
Replace: {{cssxref("$2/$1", "$1()")}}
Search: \[`([^/]+)\(\)`\]\(/en-US/docs/Web/CSS/Reference/Values/(transform-function)/\1\)
Replace: {{cssxref("$2/$1", "$1()")}}
Search: \[`([^/]+)\(\)`\]\(/en-US/docs/Web/CSS/Reference/Values/\1\)
Replace: {{cssxref("$1()")}}
Search: [`<color>`](/en-US/docs/Web/CSS/Reference/Values/color_value)
Replace: {{cssxref("<color>")}}
Search: {{CSSXref(":host_function", ":host()")}}
Replace: {{cssxref(":host()")}}
manual update for `host-context()` macro usage
Search:
for pseudo-class: \[`(:[a-z-]+(?:\(\))?)`\]\(/en-US/docs/Web/CSS/Reference/Selectors/[^)]+\)
for pseudo-element: \[`(::[a-z-]+(?:\(\))?)`\]\(/en-US/docs/Web/CSS/Reference/Selectors/[^)]+\)
Replace: {{cssxref("$1")}}
Search: \[`([^`]+)`\]\(/en-US/docs/Web/CSS/Reference/At-rules/(@media/[^)]+)\)
Replace: {{cssxref("$2")}}
Search: \[`([^`]+)\(\)`\]\(/en-US/docs/Web/CSS/Reference/Values/(color_value/[^)]+)\)
Replace: {{cssxref("$2", "$1()")}}
Search: \[`<([^>]+)>`\]\(/en-US/docs/Web/CSS/Reference/Values/[^)]+\)
Replace: {{cssxref("&1")}}
… (missing code fence in link text)
Search: \[([a-z-]+\(\))\]\(/en-US/docs/Web/CSS/Reference/Values/(transform-function/[^)]+)\)
Replace: {{cssxref("$2", "$1")}}
| of the following filter functions in a string. | ||
|
|
||
| - [`url()`](/en-US/docs/Web/CSS/Reference/Values/url_function) | ||
| - {{cssxref("url_function", "url()")}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note the next line specifies that it expects a URL, which I guess is <url> not url().
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this had tripped me up as well and I checked Blame which suggested its been url() in both places: 592bea2
| The `getSubStringLength()` method of the {{domxref("SVGTextContentElement")}} interface represents the computed length of the formatted text advance distance for a substring of text within the element. | ||
|
|
||
| Note that this method only accounts for the widths of the glyphs in the substring and any extra spacing inserted by the CSS [`letter-spacing`](/en-US/docs/Web/CSS/Reference/Properties/letter-spacing) and [`word-spacing`](/en-US/docs/Web/CSS/Reference/Properties/word-spacing) properties. Visual spacing adjustments made by the [`x`](/en-US/docs/Web/CSS/Reference/Properties/x) attribute are ignored. | ||
| Note that this method only accounts for the widths of the glyphs in the substring and any extra spacing inserted by the CSS {{cssxref("letter-spacing")}} and {{cssxref("word-spacing")}} properties. Visual spacing adjustments made by the {{cssxref("x")}} attribute are ignored. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think x is referring to Web/SVG/Reference/Attribute/x
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The original addition has a link to /en-US/docs/Web/CSS/x. See #37378.
bsmth
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two comments, but otherwise LGTM, thanks 👍🏻
|
Thanks a lot for the review! |
Description
This PR replaces CSS links in
files/en-us/web/apiwith thecssxrefmacro.Please check "Additional details" and "Motivation" in #42346 for more context.
Related issues and pull requests
cssxrefin/web/css#42346cssxrefmacro in/web/accessibility#42407cssxrefmacro in/web/html#42408cssxrefmacro inweb/http#42427cssxrefmacro inweb/mathml#42428