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 information for fullscreenchange event #4603

Merged
merged 2 commits into from
Apr 30, 2021
Merged

Conversation

SDP190
Copy link
Contributor

@SDP190 SDP190 commented Apr 29, 2021

What was wrong/why is this fix needed? (quick summary only)

Fixing wrong statement stating that the that "fullscreenchange" event is sent to the Document object when entire document is entered/exited fullscreen mode. But in real, this event never gets sent directly to document, instead when any Element in the document raises fullscreenchange event, it gets bubbled up to the Document object.

MDN URL of main page changed

https://developer.mozilla.org/en-US/docs/Web/API/Fullscreen_API

Issue number (if there is an associated issue)

#4590

Anything else that could help us review it

Read above github issue for tested prove

Fixing wrong statement stating that the that "fullscreenchange" event is sent to the Document object when entire document is entered/exited fullscreen mode. But in real, this event never gets sent directly to document, instead when any Element in the document raises fullscreenchange event, it gets bubbled up to the Document object.
@SDP190 SDP190 requested a review from a team as a code owner April 29, 2021 18:30
@SDP190 SDP190 requested review from jpmedley and removed request for a team April 29, 2021 18:30
Copy link
Contributor

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

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

LGTM; thanks @SDP190 !

@chrisdavidmills chrisdavidmills requested review from chrisdavidmills and removed request for jpmedley April 30, 2021 07:27
@github-actions
Copy link
Contributor

github-actions bot commented Apr 30, 2021

Preview URLs

Flaws

URL: /en-US/docs/Web/API/Fullscreen_API
Title: Fullscreen API
on GitHub
Flaw count: 1

  • broken_links:
    • Link points to the page it's already on

External URLs

URL: /en-US/docs/Web/API/Fullscreen_API
Title: Fullscreen API
on GitHub

(this comment was updated 2021-04-30 07:29:43.668471)

@chrisdavidmills chrisdavidmills merged commit 202f8fa into mdn:main Apr 30, 2021
wbamberg pushed a commit to wbamberg/content that referenced this pull request Apr 30, 2021
* upstream/main:
  Fix some typos: (mdn#4620)
  Fix information for fullscreenchange event (mdn#4603)
  FF89 large ArrayBuffers (mdn#4470)
  fix: HTML Validation issues (mdn#4612)
  Make warnings consistent in the JS docs (mdn#4609)
  fixed typo (mdn#4617)
  chore(deps): bump @mdn/yari from 0.4.412 to 0.4.413 (mdn#4616)
  chore(deps): bump technote-space/get-diff-action from v4.0.6 to v4.1.1 (mdn#4615)
@SDP190
Copy link
Contributor Author

SDP190 commented Apr 30, 2021

Thank you so much @chrisdavidmills

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants