Skip to content

Fix scroll snap glossary example wording#44316

Merged
hamishwillee merged 2 commits into
mdn:mainfrom
KirtiRamchandani:docs/scroll-snap-glossary-example
Jun 5, 2026
Merged

Fix scroll snap glossary example wording#44316
hamishwillee merged 2 commits into
mdn:mainfrom
KirtiRamchandani:docs/scroll-snap-glossary-example

Conversation

@KirtiRamchandani
Copy link
Copy Markdown
Contributor

Description

Updates the scroll snap glossary wording so it no longer refers to an "above carousel example" when the page only describes a carousel scenario in prose.

Motivation

The previous sentence could confuse readers because the glossary page does not include an example block above the paragraph. The revised sentence keeps the same explanation while avoiding that misleading reference.

Additional details

None.

Related issues and pull requests

Fixes #44295

Tests

  • Prettier 3.8.3 check on files/en-us/glossary/scroll_snap/index.md
  • MarkdownLint CLI 0.22.1 with MDN config on files/en-us/glossary/scroll_snap/index.md
  • git diff --check -- files/en-us/glossary/scroll_snap/index.md

@KirtiRamchandani KirtiRamchandani requested a review from a team as a code owner June 1, 2026 09:14
@KirtiRamchandani KirtiRamchandani requested review from hamishwillee and removed request for a team June 1, 2026 09:14
@github-actions github-actions Bot added Content:Glossary Glossary entries size/xs [PR only] 0-5 LoC changed labels Jun 1, 2026
Updated the definition and explanation of scroll snapping for clarity and conciseness. Removed redundant information and improved the flow of the text.
@github-actions github-actions Bot added size/s [PR only] 6-50 LoC changed and removed size/xs [PR only] 0-5 LoC changed labels Jun 5, 2026
Copy link
Copy Markdown
Collaborator

@hamishwillee hamishwillee left a comment

Choose a reason for hiding this comment

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

Thanks @KirtiRamchandani - your fix was good for the specific problem - and much appreciated.

I've rewritten it though because it had other flaws, such as including way too much CSS implementation detail. An MDN glosssary is generally intended to be concise and provide links to more detailed information for those who want it.

@hamishwillee hamishwillee merged commit a8b6ea7 into mdn:main Jun 5, 2026
8 of 9 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 5, 2026

Preview URLs (1 page)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:Glossary Glossary entries size/s [PR only] 6-50 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Scroll snap glossary — sentence about missing example

3 participants