-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[HOLD for payment 2023-01-19] Unpin tooltip text is splitting into 2 lines #14149
Comments
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open
|
@roryabraham possibly related to #12168? |
Confirmed its related to that PR by reverting it locally |
This appears to be because of a delay in rerendering the Portal's components. If we do not set a text width in |
Okay, adding a timeout before recalculating the width of the tooltip fixes this, but I think that narrows it down to the |
Job added to Upwork: https://www.upwork.com/jobs/~01757fe35604e7bf17 |
Triggered auto assignment to Contributor Plus for review of internal employee PR - @thesahindia ( |
Resolved and CP'd, but not deployed to prod. Removing the blocking label. |
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.2.52-4 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2023-01-19. 🎊 After the hold period, please check if any of the following need payment for this issue, and if so check them off after paying:
As a reminder, here are the bonuses/penalties that should be applied for any External issue:
|
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
@muttmuure can you issue payment today and also complete the checklist above, thanks |
Unassigning since I didn't do anything here. |
This was resolved internally and CP'd to prod, so no payments needed. Posted buddy check here |
@muttmuure reporting bonus is due here |
@jayeshmangwani oh yes, you're correct. I've invited you to apply. Thanks! |
@muttmuure hey , price should be $250 for reporting bonus and for the job you have invited me to is $1000 |
Should be resolved |
Please could you accept the offer? Thanks! |
@muttmuure accepted, Thanks |
@jayeshmangwani has been paid! Closing out |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Expected Result:
Unpin text should be in a single line
Actual Result:
Unpin text is splitting in 2 lines
Workaround:
unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.2.51-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:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
unpin-tooltip-text.mov
Expensify/Expensify Issue URL:
Issue reported by: @jayeshmangwani
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1673208539820589
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: