Skip to content

Add missing commas between domxref params#44039

Merged
hamishwillee merged 1 commit intomainfrom
fix-domxref-typos
May 7, 2026
Merged

Add missing commas between domxref params#44039
hamishwillee merged 1 commit intomainfrom
fix-domxref-typos

Conversation

@caugner
Copy link
Copy Markdown
Collaborator

@caugner caugner commented May 7, 2026

Description

Add missing commas between domxref params

Motivation

These were causing "must be provided" Rari issues.

Additional details

Related issues and pull requests

Part of mdn/fred#1462.

@caugner caugner requested a review from a team as a code owner May 7, 2026 19:57
@caugner caugner requested review from hamishwillee and removed request for a team May 7, 2026 19:57
@github-actions github-actions Bot added Content:WebAPI Web API docs size/s [PR only] 6-50 LoC changed labels May 7, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

Preview URLs (3 pages)

Flaws (2)

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

Found an unexpected or unresolvable flaw? Please report it here.

URL: /en-US/docs/Web/API/PreferenceObject/clearOverride
Title: PreferenceObject: clearOverride() method
Flaw count: 1

  • macros:
    • Macro domxref produces link /en-US/docs/Web/API/PreferenceObject/colorScheme which doesn't resolve

URL: /en-US/docs/Web/API/PreferenceObject/requestOverride
Title: PreferenceObject: requestOverride() method
Flaw count: 1

  • macros:
    • Macro domxref produces link /en-US/docs/Web/API/PreferenceObject/colorScheme which doesn't resolve

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 @caugner !

@hamishwillee hamishwillee merged commit cef391e into main May 7, 2026
12 checks passed
@hamishwillee hamishwillee deleted the fix-domxref-typos branch May 7, 2026 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:WebAPI Web API docs size/s [PR only] 6-50 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants