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 bug in iOS>=16.4 where the right side of a rotation-container was… #470

Merged
merged 1 commit into from
Oct 22, 2023

Conversation

sisou
Copy link
Member

@sisou sisou commented Oct 12, 2023

… not interactive

By making the rotation container not interactive, but the children yes, everything on the page can be clicked (tapped) again.

@sisou sisou self-assigned this Oct 12, 2023
@sisou sisou added the bug Something isn't working label Oct 12, 2023
Copy link
Member

@Eligioo Eligioo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested this locally on iPhone 13 iOS 16.6.1 and can confirm that this solves the issue.

@sisou sisou force-pushed the soeren/ios-fix-rotation-container branch from 0640b39 to da4da27 Compare October 22, 2023 14:26
@sisou sisou merged commit da4da27 into master Oct 22, 2023
2 checks passed
@sisou sisou deleted the soeren/ios-fix-rotation-container branch October 22, 2023 14:43
danimoh added a commit that referenced this pull request Nov 12, 2023
The Overlay component is not used in the create flow or anywhere else anymore since
USDC support was merged (PR #465), and is incompatible with the changes in the iOS
pointer-events bug fix (PR #470), where pointer-events were enabled on the overlay-frame
when they shouldn't. The incompatibility does not show directly anymore since PR #465 was
merged, but showed on the master branch before PR #465 that PR #470 was originally targeting.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants