-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
breaking(Popup): switch to Popper.js #3532
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3532 +/- ##
==========================================
- Coverage 99.82% 99.81% -0.01%
==========================================
Files 172 174 +2
Lines 2795 2725 -70
==========================================
- Hits 2790 2720 -70
Misses 5 5
Continue to review full report at Codecov.
|
* breaking(Popup): switch to Popper.js * chore(package): update react-hot-loader * update UTs * fix offset, update docs * rework docs
Is there a guide anywhere that shows how to upgrade from something like 0.84.0 with regards to Popper? The CHANGELOG says "Breaking" but offers no hints on how to fix the expected breaks. Thank you for the hard work, everyone! |
I have a feeling the "Breaking" change is the offset props being removed and the new combined offset being added. |
BREAKING CHANGES
This PR removes our positioning logic and uses
react-popper
instead.Removal of
keepInViewport
This prop was removed and will be introduced again later.
Removal of
horizontalOffset
&verticalOffset
, useoffset
nowPopper.js uses more flexible
offset
, please follow their docs.(!) Offset in Popper works differently than
horizontalOffset
&verticalOffset
:Changes in behavior
Popup
changes its position more accurate now.Before
After
Fixes #1126.
Before
After
Fixes #3226
After
Fixes #3505
Fixes #3478
Before
After