-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[$250] mWeb - Chat - Emoji picker cut off in the bottom #43692
Comments
Triggered auto assignment to @deetergp ( |
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
|
@deetergp FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors |
We think that this bug might be related to #vip-vsp |
Job added to Upwork: https://www.upwork.com/jobs/~015d3ee2fb419bfdb5 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @rayane-djouah ( |
This does not feel like a blocker to me, feel free to change that but demoting |
Please re-state the problem that we are trying to solve in this issue.Emoji picker is cut off in the bottom What is the root cause of that problem?In here , We added an extra View without any style when we added FocusTrap before. What changes do you think we should make in order to solve the problem?We can fix this bug by Remove the extra View App/src/components/Modal/BaseModal.tsx Line 256 in 361a000
What alternative solutions did you explore? (Optional) |
📣 @daledah! 📣
|
@daledah - It looks like the extra |
Offending PR: #39520 |
Looking for better proposals |
@deetergp - Can you please add |
@rayane-djouah I think this issue has been fixed in this PR |
Yes, it is fixed. We can close this one @deetergp |
Thank you for your help @daledah |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 1.4.83-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:
Expected Result:
Emoji picker should be fully rendered to the very bottom of the screen
Actual Result:
Emoji picker is cut off in the bottom
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6512117_1718293278184.Record_2024-06-13-17-17-32.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @rayane-djouahThe text was updated successfully, but these errors were encountered: