Skip to content

[Fix] Update frame on popover #664

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

Merged
merged 6 commits into from
Jul 18, 2025
Merged

[Fix] Update frame on popover #664

merged 6 commits into from
Jul 18, 2025

Conversation

chriswebb09
Copy link
Collaborator

@chriswebb09 chriswebb09 commented Jul 18, 2025

Screenshots

Line of sight between geoelements

Looks fine on iOS and Catalyst. 0.25 seemed like a reasonable amount but there is no reason to stick with it.

Before After
Simulator Screenshot - iPad Pro 13-inch (M4) - 2025-07-18 at 08 24 05 Simulator Screenshot - iPad Pro 11-inch (M4) - 2025-07-18 at 11 12 13

Browse WMS Layer

Before After
Simulator Screenshot - iPad Pro 11-inch (M4) - 2025-07-18 at 09 38 32 Simulator Screenshot - iPad Pro 11-inch (M4) - 2025-07-18 at 15 09 41
Before After
Simulator Screenshot - iPhone 16 Pro - 2025-07-18 at 11 25 26 Simulator Screenshot - iPhone 16 Pro - 2025-07-18 at 11 23 10

@chriswebb09 chriswebb09 requested review from a team, yo1995, zkline101 and CalebRas and removed request for a team and zkline101 July 18, 2025 16:41
@chriswebb09 chriswebb09 self-assigned this Jul 18, 2025
@chriswebb09 chriswebb09 marked this pull request as ready for review July 18, 2025 18:10
@chriswebb09 chriswebb09 changed the title [Fix] Possible fix for menu [Fix] Update frame on popover Jul 18, 2025
@chriswebb09 chriswebb09 requested a review from CalebRas July 18, 2025 22:11
@chriswebb09 chriswebb09 merged commit 8248660 into v.next Jul 18, 2025
1 check passed
@chriswebb09 chriswebb09 deleted the chrisw/lineofsightfix branch July 18, 2025 23:40
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.

3 participants