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

Adapt api.Performance.onresourcetimingbufferfull to new events structure #13603

Merged
merged 2 commits into from
Mar 7, 2022
Merged

Adapt api.Performance.onresourcetimingbufferfull to new events structure #13603

merged 2 commits into from
Mar 7, 2022

Conversation

queengooborg
Copy link
Collaborator

This PR adapts the resourcetimingbufferfull event of the Performance API to conform to the new events structure.

BCD PR: mdn/browser-compat-data#15206

This PR adapts the resourcetimingbufferfull event of the Performance API to conform to the new events structure.
@queengooborg queengooborg requested review from a team as code owners March 7, 2022 05:35
@queengooborg queengooborg requested review from wbamberg and removed request for a team March 7, 2022 05:35
@github-actions github-actions bot added the Content:WebAPI Web API docs label Mar 7, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Mar 7, 2022

Preview URLs

Flaws

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

URL: /en-US/docs/Web/API/Resource_Timing_API/Using_the_Resource_Timing_API
Title: Using the Resource Timing API
on GitHub
Flaw count: 2

  • broken_links:
    • Is currently http:// but can become https://
    • Is currently http:// but can become https://

URL: /en-US/docs/Web/API/Performance
Title: Performance
on GitHub
Flaw count: 3

  • macros:
    • /en-US/docs/Web/API/Window/performance redirects to /en-US/docs/Web/API/performance_property
  • broken_links:
    • Can't resolve /en-US/docs/Web/API/WorkerGlobalScope/performance
  • bad_bcd_links:
    • no explanation!

External URLs

URL: /en-US/docs/Web/API/Resource_Timing_API/Using_the_Resource_Timing_API
Title: Using the Resource Timing API
on GitHub

No new external URLs


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

No external URLs


URL: /en-US/docs/Web/API/Performance/resourcetimingbufferfull_event
Title: Performance: resourcetimingbufferfull event
on GitHub

No new external URLs

@teoli2003
Copy link
Contributor

teoli2003 commented Mar 7, 2022

Checklist I used for reviewing this event Content PR:

When onXYZ exists, for each onXYZ

  • Redirect from /Web/API/xxxx/onXYZ to /Web/API/xxxx/XYZ_event` set
  • /Web/API/xxxx/onXYZ deleted.
  • In /Web/API/xxxx/XYZ_event:
    • Front-runner YAML: Correct browser-compat front-runner key
    • Front-runner YAML: Correct title front-runner key to: Interface: XYZ event (with ticks!)
    • Front-runner YAML: No Property and Event Handler int the tag list but Event in it (more cleaning possible)
    • Check if the event bubbles. If not, no mention in article
    • Intro explain the event itself and no more just onXYZ.

      Example: "The levelchange event of the Battery Status API is fired when the battery {{domxref("BatteryManager.level","level")}} property is updated."

    • No HTML table with Bubbles, Cancelable, and Interface lines.
    • Syntax section with the structure of the template, and fix the names of event and of the onXYZ property
    • If inheritance is not only Event, there is an Event type section (two texts possible!, with
      • N/A {{InheritanceDiagram}} with the right argument
      • N/A There is an Event properties section and list the parameters, taken from the interface of the event type
    • Check that no mention of this onXYZ is a link.
    • Check that links to other onXYZ are links to the corresponding XYZ_event

On the API interface page:

  • There is no Event handlers section
  • There is an Events section (with the correct events)
  • Check that there are no links to onXYZ.

On each page of the interface and the API overview page:

  • Check that there are no links to onXYZ.
  • Check that examples uses the modern syntax for defining function in event handler (onXYZ = event => { …} )
  • Check that no {{event} macro is left.
  • Fix flaws (as much as possible); at least, make sure not to introduce new flaws.
  • Check that if an event is linked to a property value, that property documents when this event is fired.

    Example: "When its value changes, the chargingchange event is fired."

Optional:

  • Remove gremlins (non-breaking spaces, curly tick, curly double-ticks)
  • If the interface is no more experimental, remove the experimental tags/banners.

Administration:

  • Link to the BCD PR.

Copy link
Contributor

@teoli2003 teoli2003 left a comment

Choose a reason for hiding this comment

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

Looks good! One sentence that I think we can improve.

…ing_api/index.md

Co-authored-by: Jean-Yves Perrier <jypenator@gmail.com>
@teoli2003 teoli2003 merged commit c6b8023 into mdn:main Mar 7, 2022
@queengooborg queengooborg deleted the api/Performance/resourcetimingbufferfull branch March 7, 2022 14:55
wbamberg added a commit to wbamberg/content that referenced this pull request Mar 8, 2022
* upstream/main: (237 commits)
  Fixes anchor to browser compatibility table (mdn#13549)
  Fix incorrect casing of getTitlebarAreaRect (mdn#13631)
  Typo (missing s for plural) (mdn#13609)
  Fix a link (mdn#13621)
  Reduce redirects (mdn#13618)
  Adapt api.Window.onappinstalled to new events structure (mdn#13615)
  Add an example on how to list files from a `FileSystemDirectoryHandle` (mdn#13510)
  Fix a llink in the async Learn module (mdn#13623)
  Undo incorrect fix to justify-items syntax (mdn#13622)
  JS Learn: update client-side APIs, part 2 (mdn#13123)
  JS Learn area: async module rewrite (mdn#12930)
  Fix a link (mdn#13619)
  JS Learn: update client-side APIs, part 1 (mdn#13121)
  Adapt api.SharedWorker.onerror to new events structure (mdn#13599)
  Adapt api.OfflineAudioContext.oncomplete to new events structure (mdn#13568)
  Adapt MediaKeySession API to new events structure (mdn#13527)
  Adapt api.Document.onvisibilitychange to new events structure (mdn#13206)
  Change HTML table to a modern list (AudioProcessingEvent) (mdn#13606)
  Adapt api.Performance.onresourcetimingbufferfull to new events structure (mdn#13603)
  Few changes to outline. (mdn#13414)
  ...
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