Skip to content
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

fix(computeStyle): correctly calculate bottom when x="top" and `rig… #612

Merged
merged 1 commit into from Apr 15, 2018
Merged

fix(computeStyle): correctly calculate bottom when x="top" and `rig… #612

merged 1 commit into from Apr 15, 2018

Conversation

bimbiltu
Copy link

…ht` when y="left"

Fixes #585

@bimbiltu
Copy link
Author

When testing this I noticed that the reproducer in #585 had a couple of extras in it that werent causing the bug. I found this bug appears when:

  • x is set to 'top' or y is set to 'left' in computeStyle
  • the poppers offsetParent is the documentElement (did not matter where the reference was)
  • the page has enough content on it so that it scrolls

@bimbiltu
Copy link
Author

looks like the tests are failing in IE10, ill try to take a look at that later today

@bimbiltu
Copy link
Author

Fixed the unit tests, everything seems good now

@FezVrasta FezVrasta merged commit deb893f into floating-ui:master Apr 15, 2018
@FezVrasta
Copy link
Member

thanks!

@bimbiltu bimbiltu deleted the computestyle branch April 30, 2018 13:01
@bimbiltu
Copy link
Author

Hey @FezVrasta any idea when this change will be released? Id love to start using it in my project

@FezVrasta
Copy link
Member

As soon I have some free time I'll publish it, sorry for the delay

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants