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

Allow messages scrolling within edit box. #4230

Merged
merged 1 commit into from
Jul 27, 2021

Conversation

helaoutar
Copy link
Contributor

@helaoutar helaoutar commented Jul 26, 2021

Details

Fixed Issues

$ #4132

Tests

Tested on Chrome, FireFox, Safari and on Desktop.

QA Steps

  • Scenario 1:
  1. Hover over a long message
  2. Click on Edit
  3. Scroll within the edit box.

Expected:
The message should be scrollable within the edit box.

  • Scenario 2
  1. Hover over a long message
  2. Click on Edit
  3. Scroll outside the edit box.

Expected:
The message should NOT be scrollable, but the page should be.

  • Scenario 3:
  1. Hover over a long message
  2. Click on Edit
  3. Click outside the edit box.
  4. Scroll anywhere

Expected:
The message should not be scrollable neither from nor within the edit box.

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

edit.comment.chrome.mov

Mobile Web

Screen.Recording.2021-07-26.at.19.58.55.mov

Desktop

edit.demo.5.mov

iOS

Android

@helaoutar helaoutar requested a review from a team as a code owner July 26, 2021 19:05
@MelvinBot MelvinBot requested review from deetergp and removed request for a team July 26, 2021 19:05
@helaoutar helaoutar force-pushed the bugfix/4132_edit_message_scroll branch from d2a7dca to 9be3668 Compare July 26, 2021 19:06
Copy link
Contributor

@deetergp deetergp left a comment

Choose a reason for hiding this comment

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

Tests pass and code looks good.

@deetergp deetergp merged commit bd23235 into Expensify:main Jul 27, 2021
@OSBotify
Copy link
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@OSBotify
Copy link
Contributor

🚀 Deployed to staging in version: 1.0.80-3🚀

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

@OSBotify
Copy link
Contributor

🚀 Deployed to production in version: 1.0.81-4🚀

platform result
🤖 android 🤖 success ✅
🖥 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.

3 participants