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

fix(popOver): reposition PopOver when it is resized#157

Merged
Tigge merged 1 commit intoAxisCommunications:mainfrom
brian-chano:popOverposition
Oct 29, 2021
Merged

fix(popOver): reposition PopOver when it is resized#157
Tigge merged 1 commit intoAxisCommunications:mainfrom
brian-chano:popOverposition

Conversation

@brian-chano
Copy link
Copy Markdown
Collaborator

@brian-chano brian-chano commented Oct 20, 2021

  • Fixes an issue where a PopOver was incorrectly positioned after a resize

@Tigge
Copy link
Copy Markdown
Collaborator

Tigge commented Oct 21, 2021

Instead of this which seems a bit random, we instead could have an additional resize observer on the popover that could trigger a reposition. That would also solve #86 as well.

@brian-chano
Copy link
Copy Markdown
Collaborator Author

Instead of this which seems a bit random, we instead could have an additional resize observer on the popover that could trigger a reposition. That would also solve #86 as well.

Hi @Tigge, updated the code now for your review

Comment thread packages/core/src/PopOver/utils.ts Outdated
@brian-chano brian-chano force-pushed the popOverposition branch 3 times, most recently from 8222b4b to fa66ab2 Compare October 29, 2021 14:00
@brian-chano brian-chano changed the title fix(popOver): correctly position PopOver when not conditionally rendered fix(popOver): reposition PopOver when it is resized Oct 29, 2021
- Fixes an issue where a `PopOver` was incorrectly positioned after a
  resize
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants