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: room page layout #6951

Merged
merged 1 commit into from
Dec 30, 2021
Merged

Conversation

parasharrajat
Copy link
Member

@parasharrajat parasharrajat commented Dec 30, 2021

Details

Fixed Issues

$ #6929

Tests | QA Steps

  1. Open the new Room creation page.
  2. Check the button should be above the keyboard on Native platforms.

Tested On

  • Web
  • Mobile Web (we can't fix this for Mobile Safari)
  • Desktop
  • iOS
  • Android

Screenshots

Web | Desktop

image

Mobile Web

iOS

image

Android

image

@parasharrajat parasharrajat requested a review from a team as a code owner December 30, 2021 15:42
@MelvinBot MelvinBot requested review from Gonals and removed request for a team December 30, 2021 15:43
@parasharrajat
Copy link
Member Author

cc: @aldo-expensify

@aldo-expensify
Copy link
Contributor

Testing...

Copy link
Contributor

@aldo-expensify aldo-expensify left a comment

Choose a reason for hiding this comment

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

Cool, that seems to work well.

Screen Shot 2021-12-30 at 12 47 36 PM

Screenshot_20211230-124803

@github-actions
Copy link
Contributor

⚠️ ⚠️ Heads up! This pull request has the CP Staging label. ⚠️ ⚠️
Merging it will cause it to be immediately deployed to staging, even if the open StagingDeployCash deploy checklist is locked.

@aldo-expensify
Copy link
Contributor

Will merge after tests pass

@parasharrajat
Copy link
Member Author

Thanks for the screenshots.

@aldo-expensify aldo-expensify merged commit e787423 into Expensify:main Dec 30, 2021
OSBotify pushed a commit that referenced this pull request Dec 30, 2021
fix: room page layout
(cherry picked from commit e787423)
@AndrewGable
Copy link
Contributor

FYI I am fixing a bug with the cherry pick GitHub action that we found via this PR. Once we fix it we will retry this cherry pick.

OSBotify pushed a commit that referenced this pull request Dec 30, 2021
fix: room page layout
(cherry picked from commit e787423)
AndrewGable added a commit that referenced this pull request Dec 30, 2021
…ng-6951

🍒 Cherry pick PR #6951 to staging 🍒
@mvtglobally
Copy link

mvtglobally commented Dec 31, 2021

We are still seeing issue on mWeb Safari. Did the steps mean to say "We CAN'T fix it for mWeb Safari" ?

RPReplay_Final1640914353.MP4
Image.from.iOS.6.MP4

@parasharrajat
Copy link
Member Author

Sorry, mistyping. Yes, we can't fix on mobile safari @mvtglobally.

@OSBotify
Copy link
Contributor

OSBotify commented Jan 4, 2022

🚀 Deployed to staging by @aldo-expensify in version: 1.1.24-19 🚀

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

@OSBotify
Copy link
Contributor

OSBotify commented Jan 4, 2022

🚀 Deployed to production by @francoisl in version: 1.1.24-19 🚀

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

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.

5 participants