-
Notifications
You must be signed in to change notification settings - Fork 3k
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] New chat / New group animation gets stuck #2710
Comments
Triggered auto assignment to @RachCHopkins ( |
Can I just clarify - is this on the iOS app, or is it in the Web app, accessed via iOS? Are we focussing on Safari mobile, here as opposed to the iOS app? |
Oh yes! Let me fix that. |
Hmmm. Ok I am not getting the same behaviour here. If I choose a person or people whom I have an existing chat with, it's fine. Then I choose a new person I have never chatted to and it is just blank. I refresh and it's back to the chat list view. If I choose the same person a second time, it does create a new message. Video shows:
SafariTest.mp4Same behaviour on Chrome. @isagoico I'm not sure if what I see is just another version of the issue you see or something else - can you please try creating a message to the same person again, after it gets 'stuck'? And see if the issue only occurs with people you do not have an existing chat with? |
@RachCHopkins the blank screen thing is a 🐛 so this will probably go back to being weirdly broken again after we fix the other bug. |
I think it's possible @parasharrajat's issue and proposals here will also maybe address this -> #2335 Current theory is that these modals get wonky because the animations are driven in the JS and can break when we are doing too much work while transitioning. |
@marcaaron I can't replicate it. Should I close this for now if we think it will be remedied in due course? |
Maybe let's put a HOLD on it instead so we can make sure it gets resolved. I've seen this before, but it can be hard to repro. |
Sure, @marcaaron I believe that If we fix the jank behavior then this would be resolved as well. |
Issue reproducible during today's KI retests |
Triggered auto assignment to @ctkochan22 ( |
Putting a HOLD on this so we can see if Rajat's changes in review here will fix this |
Issue reproducible during today's KI retests |
1 similar comment
Issue reproducible during today's KI retests |
Issue not reproducible today during KI retests (First Week) |
Issue not reproducible today during KI retests (Second Week) |
Just to inform you that I think this was fixed in #2956. |
Issue not reproducible today during KI retests (Third Week) - Closing |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Expected Result:
New chat or New group get should work smoothly after opened from plus tab
Actual Result:
New chat or New group get stuck after opened from plus tab
Action Performed:
Workaround:
Unknown
Platform:
Where is this issue occurring?
Web
mWeb ✔️ (iOS Safari)
Android
Desktop App
Mobile Web
Version Number: the issue is repro 2 out of 5
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
Important: this is not reproducible every time. The issue is repro 2 out of 5 times.
Bug5054367_chat.mp4
Expensify/Expensify Issue URL:
View all open jobs on Upwork
The text was updated successfully, but these errors were encountered: