WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Editorial review: Document anchor-scope property #42227

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.

Sign up for GitHub

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

Jump to bottom
Open
chrisdavidmills wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Editorial review: Document anchor-scope property #42227

chrisdavidmills wants to merge 4 commits into from

Conversation

@chrisdavidmills
Copy link
Contributor

@chrisdavidmills chrisdavidmills commented Dec 4, 2025 •
edited
Loading

Description

Chrome 131 supports the anchor-scope property; see https://chromestatus.com/feature/5094192052436992. It is also supported in Safari 26, and behind a pref in Fx Nightly.

This PR documents anchor-scope.

Motivation

Additional details

Related issues and pull requests

Fixes #39295

@chrisdavidmills chrisdavidmills requested a review from a team as a code owner December 4, 2025 12:30
@chrisdavidmills chrisdavidmills requested review from estelle and removed request for a team December 4, 2025 12:30
@github-actions github-actions bot added the Content:CSS Cascading Style Sheets docs label Dec 4, 2025
@chrisdavidmills chrisdavidmills changed the title Document anchor-scope property Technical review: Document anchor-scope property Dec 4, 2025
@github-actions github-actions bot added the size/m [PR only] 51-500 LoC changed label Dec 4, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2025 •
edited
Loading

Preview URLs

Flaws (22)

Note! 1 document with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Web/CSS/Guides/Anchor_positioning/Using
Title: Using CSS anchor positioning
Flaw count: 1

  • unknown:
    • Error parsing /shared-assets/images/diagrams/css/anchor-positioning/position-area.svg: No such file or directory (os error 2)

URL: /en-US/docs/Web/CSS/Reference/Properties/anchor-name
Title: anchor-name
Flaw count: 7

  • broken_links:
    • /en-US/docs/Web/CSS/CSS_values_and_units/Value_definition_syntax#single_bar is a redirect
    • /en-US/docs/Web/CSS/dashed-ident is a redirect
    • /en-US/docs/Web/CSS/CSS_values_and_units/Value_definition_syntax#hash_mark is a redirect
  • macros:
    • Macro produces link /en-US/docs/Web/CSS/CSS_cascade/Value_processing which is a redirect
    • Macro produces link /en-US/docs/Web/CSS/CSS_cascade/Inheritance which is a redirect
    • Macro produces link /en-US/docs/Web/CSS/CSS_cascade/Value_processing which is a redirect
    • Macro produces link /en-US/docs/Web/CSS/CSS_animated_properties which is a redirect

URL: /en-US/docs/Web/CSS/Reference/Properties/anchor-scope
Title: anchor-scope
Flaw count: 8

  • broken_links:
    • /en-US/docs/Web/CSS/CSS_values_and_units/Value_definition_syntax#single_bar is a redirect
    • /en-US/docs/Web/CSS/CSS_values_and_units/Value_definition_syntax#single_bar is a redirect
    • /en-US/docs/Web/CSS/dashed-ident is a redirect
    • /en-US/docs/Web/CSS/CSS_values_and_units/Value_definition_syntax#hash_mark is a redirect
  • macros:
    • Macro produces link /en-US/docs/Web/CSS/CSS_cascade/Value_processing which is a redirect
    • Macro produces link /en-US/docs/Web/CSS/CSS_cascade/Inheritance which is a redirect
    • Macro produces link /en-US/docs/Web/CSS/CSS_cascade/Value_processing which is a redirect
    • Macro produces link /en-US/docs/Web/CSS/CSS_animated_properties which is a redirect

URL: /en-US/docs/Web/CSS/Reference/Properties/position-anchor
Title: position-anchor
Flaw count: 6

  • broken_links:
    • /en-US/docs/Web/CSS/CSS_values_and_units/Value_definition_syntax#single_bar is a redirect
    • /en-US/docs/Web/CSS/dashed-ident is a redirect
  • macros:
    • Macro produces link /en-US/docs/Web/CSS/CSS_cascade/Value_processing which is a redirect
    • Macro produces link /en-US/docs/Web/CSS/CSS_cascade/Inheritance which is a redirect
    • Macro produces link /en-US/docs/Web/CSS/CSS_cascade/Value_processing which is a redirect
    • Macro produces link /en-US/docs/Web/CSS/CSS_animated_properties which is a redirect

(comment last updated: 2025-12-08 15:41:16)

estelle
estelle reviewed Dec 4, 2025
View reviewed changes
Copy link
Member

@estelle estelle left a comment

Choose a reason for hiding this comment

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

oops, didn't realize this was in tech review... so sharing my few comments, but will wait til editorial to finish the review

files/en-us/web/css/guides/anchor_positioning/using/index.md
Comment on lines +153 to +158
font-size: 1.8rem;
color: white;
text-shadow: 1px 1px 1px black;
background-color: blue;
width: fit-content;
padding: 3px;
Copy link
Member

@estelle estelle Dec 4, 2025

Choose a reason for hiding this comment

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

suggest hiding css that isn't lesson-related.

Copy link
Contributor Author

@chrisdavidmills chrisdavidmills Dec 4, 2025

Choose a reason for hiding this comment

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

This bit is all hidden.

andruud
andruud reviewed Dec 8, 2025
View reviewed changes
Copy link

@andruud andruud left a comment

Choose a reason for hiding this comment

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

lgtm

files/en-us/web/css/reference/properties/anchor-scope/index.md Outdated

#### Result

The examples renders like so:
Copy link

@andruud andruud Dec 8, 2025

Choose a reason for hiding this comment

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

"example renders"?

or

"examples render"

Copy link
Contributor Author

@chrisdavidmills chrisdavidmills Dec 8, 2025

Choose a reason for hiding this comment

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

Oops. Fixed.

files/en-us/web/css/reference/properties/anchor-scope/index.md Outdated

The examples renders like so:

{{ EmbedLiveSample("basic_usage", "100%", "225") }}
Copy link

@andruud andruud Dec 8, 2025

Choose a reason for hiding this comment

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

I don't know how MDN works, but naively I'd assume this embeds the exact same example as on line 133.

Copy link
Contributor Author

@chrisdavidmills chrisdavidmills Dec 8, 2025

Choose a reason for hiding this comment

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

Yeah, this should be "comparing-values". Fixed.

@chrisdavidmills
Copy link
Contributor Author

chrisdavidmills commented Dec 8, 2025

Thanks for the review, @andruud; I'll move this one on to the editorial review stage now.

@chrisdavidmills chrisdavidmills changed the title Technical review: Document anchor-scope property Editorial review: Document anchor-scope property Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@estelle estelle Awaiting requested review from estelle

1 more reviewer

@andruud andruud andruud left review comments

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

Content:CSS Cascading Style Sheets docs size/m [PR only] 51-500 LoC changed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

CSS Anchor Positioning: anchor-scope documentation missing

3 participants

@chrisdavidmills @estelle @andruud