Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

Adds scheduler.yield callout in the Optimize Long Tasks guide. #10362

Merged
merged 3 commits into from
Aug 26, 2023

Conversation

malchata
Copy link
Member

Changes proposed in this pull request:

@malchata malchata added P1 A high priority task. This task *must* be completed soon. content update for issues that do not require new content (only for updates to existing content) labels Aug 26, 2023
@malchata malchata self-assigned this Aug 26, 2023
@netlify
Copy link

netlify bot commented Aug 26, 2023

Deploy Preview for web-dev-staging ready!

Name Link
🔨 Latest commit 3c730c2
🔍 Latest deploy log https://app.netlify.com/sites/web-dev-staging/deploys/64ea52085bbf6a0008e53df4
😎 Deploy Preview https://deploy-preview-10362--web-dev-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@@ -349,9 +349,13 @@ This is a simplistic example of how `postTask()` can be used. It's possible to i
[`postTask()` is not supported in all browsers](https://caniuse.com/mdn-api_scheduler_posttask). You can use feature detection to see if it's available, or consider using [a polyfill](https://www.npmjs.com/package/scheduler-polyfill).
{% endAside %}

### Built-in yield with continuation
### Built-in yield with continuation with `scheduler.yield`
Copy link
Member

Choose a reason for hiding this comment

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

Double “with” if can think of a better way of putting it.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah yeah, good catch. Will fix it with an inline commit.

@malchata malchata merged commit 817ded8 into main Aug 26, 2023
9 of 10 checks passed
@malchata malchata deleted the adds-scheduler-yield-callout branch August 26, 2023 19:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
content update for issues that do not require new content (only for updates to existing content) P1 A high priority task. This task *must* be completed soon.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants