-
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 https://github.com/Expensify/App/issues/27672] [$500] Web - App becomes unresponsive on selecting and pasting same emoji in composer #28035
Comments
ProposalPlease re-state the problem that we are trying to solve in this issue.App becomes unresponsive on selecting and pasting same emoji in composer What is the root cause of that problem?When we get the cursor position here
This loop run infinite, because the two values are the same, and no break for this App/src/libs/ComposerUtils/index.js Lines 39 to 42 in a240941
while (str1[str1.length - 1 - i] === str2[str2.length - 1 - i]) { i++}
// The infinite value happend when i >= str.length
str[str.length - 1 - i]
str[100 - 1 - 100]
str[-1] = undefined
while (undefined === undefined) { i++} // infinite loop What changes do you think we should make in order to solve the problem?We need to break the loop when comparison end by adding this condition to while What alternative solutions did you explore? (Optional)N/A |
Job added to Upwork: https://www.upwork.com/jobs/~018a748087cda69421 |
Triggered auto assignment to @Christinadobrzyn ( |
Bug0 Triage Checklist (Main S/O)
|
Triggered auto assignment to @bfitzexpensify ( |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @narefyev91 ( |
I think this issue may be will not be reproduced after fix #27672 |
Hello @kbecciv can i work on these issues even if didnt submit proposal in upwork, is there any bounty paid? |
📣 @rameshvoodi! 📣
|
@Christinadobrzyn hey - can you please put on hold this one - to wait a potential fix from this issue #27672 |
Updated the title to hold for #27672 |
#27672 has been merged, so we should hopefully be good to check back on this early next week |
@narefyev91, @bfitzexpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Waiting for #27672 to be deployed. |
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:
App should not become unresponsive by any action
Actual Result:
App becomes unresponsive when we paste emoji text on same emoji selected in composer
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.73.0
Reproducible in staging?: y
Reproducible in production?: y
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: Any additional supporting documentation
unresponsive.app.on.selecting.emoji.and.pasting.emoji.text.over.it.mp4
Recording.4695.mp4
Expensify/Expensify Issue URL:
Issue reported by: @dhanashree-sawant
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1695225569076869
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: