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.

Sign up for GitHub

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

Jump to bottom

Popover2 will cause the page to scroll up unexpectedly #4981

Closed
goncaloc1 opened this issue Oct 22, 2021 · 1 comment · Fixed by #4982
Closed

Popover2 will cause the page to scroll up unexpectedly #4981

goncaloc1 opened this issue Oct 22, 2021 · 1 comment · Fixed by #4982
Assignees
@adidahiya
Labels
P1 Package: popover2 Type: bug

Comments

@goncaloc1
Copy link

goncaloc1 commented Oct 22, 2021 •
edited
Loading

Environment

  • Package version(s): @blueprintjs/core: 3.51.0, @blueprintjs/popover2: 0.12.5
  • Operating System: Any
  • Browser name and version: Chrome, Firefox ESR (both latest)

Steps to reproduce

  1. On a page with vertical scrollbar, scroll down;
  2. Hover an item with Popover2 component.

Actual behavior

Popover2 will be displayed but the page will scroll up automatically and unexpectedly.

popover2_bug.mp4

Possible solution

Setting usePortal Popover2 property to false will fix the issue but that's just a workaround.
@adidahiya adidahiya self-assigned this Oct 22, 2021
@adidahiya
Copy link
Contributor

This is a major regression, I'm looking into this ASAP

This was referenced Oct 22, 2021
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adidahiya adidahiya

Labels
P1 Package: popover2 Type: bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[core] fix(Overlay): fix scroll regression for popovers palantir/blueprint
2 participants
@adidahiya @goncaloc1