RocketChat / Rocket.Chat.Electron

Official OSX, Windows, and Linux Desktop Clients for Rocket.Chat
https://rocket.chat/
MIT License
1.59k stars 704 forks source link

Sidebar does not highlight workspace with new messages #2844

Open paulchen opened 8 months ago

paulchen commented 8 months ago

Search before asking

Operating System

Operating System Version

Windows 10

It happens on the web browser too?

No, it just happens on the Desktop app

Rocket.Chat Desktop App Version

3.9.13

Rocket.Chat Server Version

6.6.0

Describe the bug

When I'm highlighted, the taskbar icon changes appropriately:

OdhVSsAoeQxqmTMN

The sidebar of the desktop app also highlights the workspace where I'm highlighted:

Q76Rs6i0nYJ9S7TX

When there are only new messages that do not highlight me, the taskbar icon changes as well:

YfPqEskPZzY4oT0m

However, the icon in the sidebar does not change:

0EwXHOY76C1xHxyf

As a consequence, I need to cycle through all workspaces to find out where there is the new message.

How to Reproduce

  1. Set up at least two workspaces in the desktop app.
  2. Configure the notification settings for the first workspace to not highlight you for every new message.
  3. Open the second workspace and keep it open.
  4. With another user, write a message in the first workspace that will be visible to the initial user.

Describe your Expected behavior

Both taskbar and sidebar will change in order to guide me to the new message in the first workspace.

Anything else

I am pretty sure that there was a time when this worked as expected. However, it stopped working some time ago.

Are you willing to submit a code contribution?

jeanfbrito commented 8 months ago

Hey @paulchen I tested it and worked for me. This happens in how many servers? Its all the time?

paulchen commented 8 months ago

Hey @paulchen I tested it and worked for me. This happens in how many servers? Its all the time?

Yes, it's all the time.

I was just able to reproduce it using a clean Snap-based client installation on Arch Linux that connects to two servers.

Let me clarify that by "sidebar" I mean the list of servers, not the list of channels/groups on one of the servers.

jeanfbrito commented 8 months ago

Hey @paulchen I tested it and worked for me. This happens in how many servers? Its all the time?

Yes, it's all the time.

I was just able to reproduce it using a clean Snap-based client installation on Arch Linux that connects to two servers.

Let me clarify that by "sidebar" I mean the list of servers, not the list of channels/groups on one of the servers.

But it wasn't on Windows? I don't get this clean Snap-based client installation on Arch Linux that. I cant say if I don't understood how to reproduce or if it don't happens to me. Could you provide some screen record showing it happening?

paulchen commented 8 months ago

But it wasn't on Windows? I don't get this clean Snap-based client installation on Arch Linux that.

Usually I only use the Electron client on Windows 10. That's where I initially observed the problem. By trying a clean installation using the Snap image, I wanted to verify the following things:

I think I can provide a screen recording, but it will take me some time. I need to take care not to share any sensitive data.

jeanfbrito commented 8 months ago

No problem. I will try to record my test tomorrow so you can say if I'm testing right. You disable notifications on desktop on server 1, change to server 2 and send a message using another user on server 1, right?

jeanfbrito commented 8 months ago

Hey, I released a new version 3.9.14 now, I found a bug that could lead to this issue. So please test this latest version and tell me if the problem persists.

paulchen commented 8 months ago

Hey, I released a new version 3.9.14 now, I found a bug that could lead to this issue. So please test this latest version and tell me if the problem persists.

Unfortunately, the problem still persists.

I created some screen recordings to showcase the situation:

In the first video, we see the initial situation:

https://github.com/RocketChat/Rocket.Chat.Electron/assets/404840/a5e3068d-4431-4e41-910c-4ecd5be3b2a5

In the second video, the following happens:

In summary, the "1"s lead the user "paulchen" step-by-step to the message where they have been highlighted.

https://github.com/RocketChat/Rocket.Chat.Electron/assets/404840/f09b5330-bf74-4b9c-a45c-a4f18c351b34

In the third video, the following happens:

The main difference to the second video is that "test1" does not highlight "paulchen" in their message. Instead, they just write a simple message to a channel where "paulchen" is joined. After "test1" submits their message, the tray icon will show a dot denoting a new message, but the server list of the desktop app does not lead "paulchen" to the server and the channel where the message has been written.

In my opinion, there should be a dot in the server list for any server where a new message has been written. Maybe the current behaviour is intentional, but as stated in the initial report, I think some time ago such a dot was shown. But maybe I'm wrong.

https://github.com/RocketChat/Rocket.Chat.Electron/assets/404840/85f2e289-90cc-4568-acc3-f5e635ddb062