Skip to content
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.

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

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

Merged
merged 1 commit into from
Aug 11, 2022
Merged

Conversation

de-oz
Copy link
Contributor

@de-oz de-oz commented Aug 11, 2022

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

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

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
* 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
* 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
Labels
Content:WebAPI Web API docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants