Skip to content
This repository was archived by the owner on Mar 4, 2020. It is now read-only.

fix(Popper): patch popper to use the correct window instance#2028

Merged
jurokapsiar merged 8 commits intomasterfrom
fix/popper-window
Oct 17, 2019
Merged

fix(Popper): patch popper to use the correct window instance#2028
jurokapsiar merged 8 commits intomasterfrom
fix/popper-window

Conversation

@jurokapsiar
Copy link
Contributor

@jurokapsiar jurokapsiar commented Oct 15, 2019

Popper is using global window when calling requestAnimationFrame. Patch it to use the correct window.
An issue will be created in popper to fix this properly.

@codecov
Copy link

codecov bot commented Oct 15, 2019

Codecov Report

Merging #2028 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2028      +/-   ##
==========================================
+ Coverage   75.83%   75.84%   +0.01%     
==========================================
  Files         160      160              
  Lines        5569     5573       +4     
  Branches     1611     1611              
==========================================
+ Hits         4223     4227       +4     
  Misses       1332     1332              
  Partials       14       14
Impacted Files Coverage Δ
packages/react/src/lib/positioner/Popper.tsx 88.46% <100%> (+0.96%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 16db5cd...cdbb148. Read the comment docs.

@layershifter layershifter added 🚀 ready for review 🧰 fix Introduces fix for broken behavior. labels Oct 17, 2019
@jurokapsiar jurokapsiar merged commit 6db9dfe into master Oct 17, 2019
@jurokapsiar jurokapsiar deleted the fix/popper-window branch October 17, 2019 14:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

🧰 fix Introduces fix for broken behavior. 🚀 ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants