Skip to content

Comments

Fix incorrect indefinite article usage (part #3)#19409

Merged
wbamberg merged 1 commit intomdn:mainfrom
de-oz:de-oz-patch
Aug 11, 2022
Merged

Fix incorrect indefinite article usage (part #3)#19409
wbamberg merged 1 commit intomdn:mainfrom
de-oz:de-oz-patch

Conversation

@de-oz
Copy link
Contributor

@de-oz de-oz commented Aug 11, 2022 •
edited
Loading

Summary

Changed "an" to "a" articles before consonant sounds.

Motivation

Supporting details

Related issues

Metadata

  • Adds a new document
  • Rewrites (or significantly expands) a document
  • Fixes a typo, bug, or other error

@de-oz de-oz requested a review from a team as a code owner August 11, 2022 12:49
@de-oz de-oz requested review from wbamberg and removed request for a team August 11, 2022 12:49
@github-actions github-actions bot added the Content:WebAPI Web API docs label Aug 11, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Aug 11, 2022

Preview URLs

Flaws

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

URL: /en-US/docs/Web/API/Element/keyup_event
Title: Element: keyup event
on GitHub
Flaw count: 2

  • macros:
    • /en-US/docs/Web/API/KeyboardEvent/locale does not exist
  • broken_links:
    • Can't resolve /en-US/docs/Web/API/Document/keyup_event

URL: /en-US/docs/Web/API/Element/dblclick_event
Title: Element: dblclick event
on GitHub
Flaw count: 3

  • macros:
    • /en-US/docs/Web/API/MouseEvent/mozPressure does not exist
    • /en-US/docs/Web/API/HTMLElement/pointerdown_event redirects to /en-US/docs/Web/API/Element/pointerdown_event
    • /en-US/docs/Web/API/HTMLElement/pointerup_event redirects to /en-US/docs/Web/API/Element/pointerup_event

URL: /en-US/docs/Web/API/Element/keypress_event
Title: Element: keypress event
on GitHub
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/KeyboardEvent/locale does not exist

URL: /en-US/docs/Web/API/Element/keydown_event
Title: Element: keydown event
on GitHub
Flaw count: 2

  • macros:
    • /en-US/docs/Web/API/KeyboardEvent/locale does not exist
  • broken_links:
    • Can't resolve /en-US/docs/Web/API/Document/keydown_event

URL: /en-US/docs/Web/API/Element/auxclick_event
Title: Element: auxclick event
on GitHub
Flaw count: 4

  • macros:
    • /en-US/docs/Web/API/MouseEvent/mozPressure does not exist
    • /en-US/docs/Web/API/HTMLElement/pointerdown_event redirects to /en-US/docs/Web/API/Element/pointerdown_event
    • /en-US/docs/Web/API/HTMLElement/pointerdown_event redirects to /en-US/docs/Web/API/Element/pointerdown_event
    • /en-US/docs/Web/API/HTMLElement/pointerup_event redirects to /en-US/docs/Web/API/Element/pointerup_event

External URLs

URL: /en-US/docs/Web/API/Element/keyup_event
Title: Element: keyup event
on GitHub

No new external URLs

URL: /en-US/docs/Web/API/Element/dblclick_event
Title: Element: dblclick event
on GitHub

No new external URLs

URL: /en-US/docs/Web/API/Element/gotpointercapture_event
Title: Element: gotpointercapture event
on GitHub

No new external URLs

URL: /en-US/docs/Web/API/Element/blur_event
Title: Element: blur event
on GitHub

No new external URLs

URL: /en-US/docs/Web/API/Element/keypress_event
Title: Element: keypress event
on GitHub

No new external URLs

URL: /en-US/docs/Web/API/Element/focus_event
Title: Element: focus event
on GitHub

No new external URLs

URL: /en-US/docs/Web/API/Element/keydown_event
Title: Element: keydown event
on GitHub

No new external URLs

URL: /en-US/docs/Web/API/Element/auxclick_event
Title: Element: auxclick event
on GitHub

No new external URLs

wbamberg
wbamberg approved these changes Aug 11, 2022
View reviewed changes
Copy link
Collaborator

@wbamberg wbamberg left a comment

Choose a reason for hiding this comment

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

👍 nice, thank you!

@wbamberg wbamberg merged commit 94ab801 into mdn:main Aug 11, 2022
@de-oz de-oz deleted the de-oz-patch branch August 11, 2022 15:17
wbamberg added a commit that referenced this pull request Aug 31, 2022
@de-oz @wbamberg
Correction for the formal syntax not found issue - 02 (#19416)
230360d 
* Fix incorrect indefinite article usage (part #3) (#19409)

* Revert "Fix incorrect indefinite article usage (part #3) (#19409)"

This reverts commit 4e694e0.

* Fix for the formal syntax

* Fix for the formal syntax

* Fix code review comments

* Consistently indent with 2 spaces

Co-authored-by: de-oz <[email protected]>
Co-authored-by: Will Bamberg <[email protected]>
goshdarnheck pushed a commit to goshdarnheck/content that referenced this pull request Sep 7, 2022
@de-oz @wbamberg @goshdarnheck
Correction for the formal syntax not found issue - 02 (mdn#19416)
1ec719a 
* Fix incorrect indefinite article usage (part mdn#3) (mdn#19409)

* Revert "Fix incorrect indefinite article usage (part mdn#3) (mdn#19409)"

This reverts commit 4e694e0.

* Fix for the formal syntax

* Fix for the formal syntax

* Fix code review comments

* Consistently indent with 2 spaces

Co-authored-by: de-oz <[email protected]>
Co-authored-by: Will Bamberg <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wbamberg wbamberg wbamberg approved these changes

Assignees

No one assigned

Labels

Content:WebAPI Web API docs

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@de-oz @wbamberg