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 iPad overlay / click away handler issues #2525

Merged
merged 4 commits into from
Apr 22, 2021

Conversation

marcaaron
Copy link
Contributor

@marcaaron marcaaron commented Apr 22, 2021

Details

Simplifies some logic around the "click away handler" which is necessary for "wide screen" views to dismiss RH Modals.

Fixed Issues

Fixes #2127

Tests

QA Steps

  1. Open settings or profile pages
  2. Verify the area to the left of the modal turns greyed out
  3. Click/tap the area to the left of the modal and verify the modal is dismissed
  4. Verify all links on the settings page can be accessed

Tested On

  • Web
  • Mobile Web (N/A)
  • Desktop
  • iOS (iPad)
  • Android (Tablet)

Screenshots

Web

Mobile Web

Desktop

iOS

Android

@marcaaron marcaaron self-assigned this Apr 22, 2021
@marcaaron marcaaron marked this pull request as ready for review April 22, 2021 02:02
@marcaaron marcaaron requested a review from a team as a code owner April 22, 2021 02:02
@MelvinBot MelvinBot requested review from ctkochan22 and removed request for a team April 22, 2021 02:02
@ctkochan22 ctkochan22 merged commit 65e6000 into main Apr 22, 2021
@ctkochan22 ctkochan22 deleted the marcaaron-fixSomeIpadIssues branch April 22, 2021 20:22
@OSBotify
Copy link
Contributor

OSBotify commented May 8, 2021

🚀 Deployed to production in version: 1.0.39-5🚀

platform result
🤖 android 🤖 failure ❌
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

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.

iOS / iPad - New Chat - No overlay in conversation area when right panel is open
3 participants