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

✨add defer prop for requestIdleCallback in react-import-remote #561

Merged
merged 1 commit into from
Mar 8, 2019

Conversation

cartogram
Copy link
Contributor

@cartogram cartogram commented Mar 7, 2019

Part of https://github.com/Shopify/web/issues/10479

This PR adds a defer prop to the ImportRemote component that when set to Idle enum, loads the request with requestIdelCallback.

Some shared types have been moved to the basic async package from react-async.

TODO

  • Add tests? Will first do a separate PR to add requestIdleCallback to jest-dom-mocks.
  • Update Readme. Will wait for general approval of this first.

Copy link
Member

@lemonmade lemonmade left a comment

Choose a reason for hiding this comment

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

Looks great 👍

@cartogram cartogram force-pushed the react-import-remote__ricb branch 5 times, most recently from 307d3ab to 0edd490 Compare March 8, 2019 17:12
react-shortcuts: Remove prop-types as a dependency

PR #295 refactored react-shortcuts to use the modern context API which
means prop-types is no longer needed

Readme updates
@cartogram cartogram merged commit 88bbc4f into master Mar 8, 2019
@cartogram cartogram deleted the react-import-remote__ricb branch March 8, 2019 18:56
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