Skip to content

Fix scroll-state queries snapped ambiguity#40354

Merged
estelle merged 1 commit intomdn:mainfrom
chrisdavidmills:fix-scroll-state-queries-ambiguity
Jul 15, 2025
Merged

Fix scroll-state queries snapped ambiguity#40354
estelle merged 1 commit intomdn:mainfrom
chrisdavidmills:fix-scroll-state-queries-ambiguity

Conversation

@chrisdavidmills
Copy link
Copy Markdown
Contributor

@chrisdavidmills chrisdavidmills commented Jul 14, 2025

Description

In #40290, @LukeAtlas22 points out that our scroll-state queries docs contain an ambiguity. It says:

snapped: Queries whether a container is, or will be, snapped to a scroll snap container ancestor along a given axis.

But the evaluation occurs only on the scrollsnapchanging event, which fires when an element will be snapped, not when it is snapped. Therefore, the above wording needs an update.

This PR provides that update.

Motivation

Additional details

Related issues and pull requests

Fixes #40290.

@chrisdavidmills chrisdavidmills requested a review from a team as a code owner July 14, 2025 12:22
@chrisdavidmills chrisdavidmills requested review from estelle and removed request for a team July 14, 2025 12:22
@github-actions github-actions bot added Content:CSS Cascading Style Sheets docs size/s [PR only] 6-50 LoC changed labels Jul 14, 2025
@github-actions
Copy link
Copy Markdown
Contributor

Preview URLs

Flaws (12)

Note! 2 documents with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Web/CSS/@container
Title: @container
Flaw count: 12

  • broken_links:
    • Can't resolve /en-US/docs/Web/CSS/block-contents
    • Can't resolve /en-US/docs/Web/CSS/size-feature
    • Can't resolve /en-US/docs/Web/CSS/scroll-state-feature
    • Can't resolve /en-US/docs/Web/CSS/function-token
    • Can't resolve /en-US/docs/Web/CSS/any-value
    • and 7 more flaws omitted

Copy link
Copy Markdown
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.

Thanks! 🎉

@estelle estelle merged commit 93ca748 into mdn:main Jul 15, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:CSS Cascading Style Sheets docs size/s [PR only] 6-50 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ambiguity in Documentation: scoll state queries and scroll snap events.

3 participants