-
Notifications
You must be signed in to change notification settings - Fork 23
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/reactist
version: v15.0.0
Metadata
Metadata
Assignees
Labels
No labels