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

Web -Chat -Unable to scroll large text in expanded compose box with large text #11885

Closed
kavimuru opened this issue Oct 15, 2022 · 18 comments
Closed
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors

Comments

@kavimuru
Copy link

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Action Performed:

  1. Go to URL https://staging.new.expensify.com/
  2. Login with any account
  3. Navigate to any chat
  4. Add large text
  5. Make a field extension

Expected Result:

Large texts must be scrolled in the extension field

Actual Result:

Unable to scroll large text in open format in compose box

Workaround:

unknown

Platform:

Where is this issue occurring?

  • Web
  • Desktop

Version Number: 1.2.16-2
Reproducible in staging?: y
Reproducible in production?: y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Bug5778195_Recording__1554.mp4

Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:

View all open jobs on GitHub

@melvin-bot
Copy link

melvin-bot bot commented Oct 15, 2022

Triggered auto assignment to @techievivek (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@techievivek
Copy link
Contributor

This indeed is a problem. Once we expand the compose box, we can no longer scroll.

@melvin-bot melvin-bot bot removed the Overdue label Oct 18, 2022
@puneetlath puneetlath added the Bug Something is broken. Auto assigns a BugZero manager. label Oct 19, 2022
@techievivek
Copy link
Contributor

Passing it to external contributors to have a look. My guess is it is some styling issue.

@techievivek techievivek added the External Added to denote the issue can be worked on by a contributor label Oct 20, 2022
@melvin-bot
Copy link

melvin-bot bot commented Oct 20, 2022

Triggered auto assignment to @sonialiap (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@melvin-bot
Copy link

melvin-bot bot commented Oct 20, 2022

Triggered auto assignment to Contributor-plus team member for initial proposal review - @sobitneupane (External)

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Oct 20, 2022
@melvin-bot
Copy link

melvin-bot bot commented Oct 20, 2022

Triggered auto assignment to @luacmartins (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@melvin-bot melvin-bot bot changed the title Web -Chat -Unable to scroll large text in expanded compose box with large text [$250] Web -Chat -Unable to scroll large text in expanded compose box with large text Oct 20, 2022
@thesahindia
Copy link
Member

Will be fixed by #11608

@techievivek techievivek changed the title [$250] Web -Chat -Unable to scroll large text in expanded compose box with large text [HOLD] Web -Chat -Unable to scroll large text in expanded compose box with large text Oct 20, 2022
@techievivek
Copy link
Contributor

I am keeping it on HOLD because I can see that the PR above will fix this issue. Sorry for the pings.

@melvin-bot
Copy link

melvin-bot bot commented Oct 24, 2022

@techievivek Whoops! This issue is 2 days overdue. Let's get this updated quick!

@melvin-bot melvin-bot bot added the Overdue label Oct 24, 2022
@techievivek
Copy link
Contributor

#11885 (comment) We already have a fix for it so once it is deployed we will be good to close this issue.

@melvin-bot melvin-bot bot removed the Overdue label Oct 24, 2022
@mvtglobally
Copy link

Issue not reproducible during KI retests. (First week)

@melvin-bot melvin-bot bot added the Overdue label Oct 26, 2022
@techievivek
Copy link
Contributor

Not overdue. We have put it on hold because we have already fixed this here.

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Oct 26, 2022
@techievivek
Copy link
Contributor

Not overdue.

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Oct 31, 2022
@techievivek
Copy link
Contributor

Not overdue, the issue is on hold as we have a fix here.

@melvin-bot melvin-bot bot removed the Overdue label Oct 31, 2022
@JmillsExpensify
Copy link

We should assign someone from Bug Zero to test this one now that #11437 is merged.

@JmillsExpensify JmillsExpensify changed the title [HOLD] Web -Chat -Unable to scroll large text in expanded compose box with large text Web -Chat -Unable to scroll large text in expanded compose box with large text Nov 1, 2022
@mvtglobally
Copy link

Issue not reproducible during KI retests. (Second week)

@melvin-bot melvin-bot bot added the Overdue label Nov 3, 2022
@JmillsExpensify
Copy link

@techievivek I think we're good to close this issue now, right?

@techievivek
Copy link
Contributor

Yep, We are safe to close this now.

@melvin-bot melvin-bot bot removed the Overdue label Nov 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors
Projects
None yet
Development

No branches or pull requests

9 participants