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(ui5-popover): restrict arrow going out of bounds #4599

Merged
merged 4 commits into from
Jan 21, 2022

Conversation

georgimkv
Copy link
Contributor

Part of #4581

The arrow of the Popover is now restricted within a range along each
dimension. The border-radius of the Popover in the different themes is
considered.

This solves the issue where the Popover may be opened by a
small element placed at the corners of the viewport.

Part of #4581

The arrow of the Popover is now restricted within a range along each
dimension.  The border-radius of the Popover in the different themes is
considered.

This solves the issue where the Popover may be opened by a
small element placed at the corners of the viewport.
@georgimkv georgimkv requested a review from a team January 20, 2022 12:18
@georgimkv georgimkv marked this pull request as ready for review January 20, 2022 12:18
@georgimkv georgimkv changed the title fix(ui5-popover): fixed arrow going out of bounds fix(ui5-popover): restrict arrow going out of bounds Jan 20, 2022
@ilhan007 ilhan007 merged commit 41239a8 into master Jan 21, 2022
@ilhan007 ilhan007 deleted the popover_arrow_fix branch March 24, 2022 16:22
georgimkv added a commit that referenced this pull request Mar 25, 2022
Improved logic from #4599 for clamping the arrow's placement.

Fixes #4797
georgimkv added a commit that referenced this pull request Mar 28, 2022
…ers (#4960)

Improved logic from #4599 for clamping the arrow's placement.

Fixes #4797
ilhan007 pushed a commit that referenced this pull request Mar 30, 2022
…ers (#4960)

Improved logic from #4599 for clamping the arrow's placement.

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

Successfully merging this pull request may close these issues.

3 participants