-
Notifications
You must be signed in to change notification settings - Fork 24
Open
Description
🐛 Bug report
Current behavior
It's possible for a consuming app to also have React Focus Lock installed, and if the version used in Reactist can't be deduped by npm/yarn, multiple versions can end up being bundled into the app, resulting in the two fighting for focus: https://github.com/theKashey/react-focus-lock#warning
Instead, declare it as a peer dependency so that npm/yarn can properly warn users if there is a mismatch.
Steps to reproduce the bug
Discovered while testing https://github.com/Doist/twist-web/pull/5071. If Twist is using v2.5.2, and 2.9.1 is installed with Reactist, some tests will fail.
Possible solutions
Declare React Focus Lock as a peer dependency
Environment
@doist/reactistversion: v15.0.0
Metadata
Metadata
Assignees
Labels
No labels