-
Notifications
You must be signed in to change notification settings - Fork 830
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
[Bug]: Mattermost desktop app doesn't refresh UI when opened from notification #3132
Comments
We are facing the same issue, and we haven't identified a consistent pattern to reproduce it.
|
I've seen this before - but have never been able to reproduce. Can someone take a video for me just so I can see exactly what the steps look like? |
issue.mp4Here is the video with the error. It was very difficult to reproduce. |
@ZarenOFF Thanks for the video! No worries about the blur, so it looks like we're seeing a mention badge in the Desktop App at the top, but as said the view literally isn't navigating. One more request - can you turn on Debug logging in the Settings modal, reproduce one more time and send me the logs? (you may want to sanitize those before sending as well) |
@devinbinnie Here is the full log with debug enabled. The issue occurred when the message arrived at 15:34:51. |
Thanks @ZarenOFF. I'll have to investigate and get back to you. |
@ZarenOFF I spent some time trying to reproduce this but unfortunately got nowhere. I have a couple theories as to what might be going on, but unfortunately nothing I can act on since I can't get reproduce the issue. My leading theory is: The tab has gone to sleep (which is why we see websocket reconnecting messages) and is not waking up correctly until some kind of user interaction happens, which is why it's not navigating until you click somewhere. It's probably happening intermittently due to some race condition between methods that unminimize/unhide the app and navigate to the correct channel. I think there might be a way to force that to happen, but I've been unsuccessful doing so so far. I'll hopefully be able to spend more time on it eventually, but right now I don't have the time to focus on it. Thanks for all the feedback :) |
@devinbinnie I would also like to add that we are using HAProxy as a reverse proxy for Mattermost, which might be important. Today, we slightly adjusted the HAProxy configuration based on this article, and we will see if it has any effect. |
@devinbinnie Finally, we were able to find the reason leading to the error. Here are the exact steps to reproduce the problem:
issue2.mp4It is important not to minimize or close Mattermost, this only occurs when another window is on top of it! This issue is reproducible on Windows 10 22H2, however, our tests on Windows 11 22H3 did not reveal this issue. |
Thanks for the video @ZarenOFF, unfortunately given your exact steps I am still unable to reproduce your issue. This is starting to feel like less of a Desktop App issue and more than likely an issue on your machine. Is this consistently reproducible across multiple Windows 10 machines? Do you have any window management software on those machines? |
@devinbinnie Additionally, the problem only appears on Windows 10. Everything works fine on Windows 11. Tests were conducted on several computers with different hardware. If it matters: Testing was done only on NVidia graphics cards with Intel processors. |
@Reydan46 Thanks, that might be helpful, I have been mostly testing in VMs for Windows 10 (all my other Windows machines are Windows 11). I will see if I can reproduce given those conditions. |
Checks before filing an issue
Mattermost Desktop Version
5.9.0
Operating System
Windows 10 x64 22H2
Mattermost Server Version
9.11
Steps to reproduce
Expected behavior
When opening the app from a notification, the UI should immediately navigate to the chat with the new message and display it without requiring an additional click.
Observed behavior
The app opens but remains in its previous state, and the new message and chat are not visible until the user clicks somewhere in the app window.
Log Output
Additional Information
It's important to note that this issue is intermittent and difficult to reproduce consistently. I haven't been able to identify a specific pattern or set of conditions that consistently lead to this behavior.
The text was updated successfully, but these errors were encountered: