Skip to content
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.

Sign up for GitHub

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

Jump to bottom

No notifications for activity in chats you're not looking at while the app is focused #15655

Closed
Pointedstick opened this issue Nov 5, 2020 · 1 comment
Closed

No notifications for activity in chats you're not looking at while the app is focused #15655

Pointedstick opened this issue Nov 5, 2020 · 1 comment
Labels
A-Notifications P1 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Product More input needed from the Product team

Comments

@Pointedstick
Copy link

Pointedstick commented Nov 5, 2020 •
edited
Loading

Description

When the app is focused, it doesn't send desktop notifications for chats that you're not currently viewing. As a result, while you;re using the app, if people send messages to other chats that you're not actively looking at, or send you a direct message, you'll miss them.

Steps to reproduce

  • Connect to two chats
  • While the app is still focused and you're in one of the rooms, have someone send messages to the other room, or send you a direct message

At this point, you will mess the messages, because the app will not send a desktop notification about it when it's focused. This is particularly bad when you miss a direct message.

If the app had instead been running in the background, you would instead receive a notification and not mess the message.

Version information

  • Platform: desktop
  • OS: Linux: openSUSE Tumbleweed
  • Version: 1.7.10
@dbkr dbkr added X-Needs-Product More input needed from the Product team A-Notifications P1 S-Minor Impairs non-critical functionality or suitable workarounds exist labels Nov 6, 2020
@t3chguy
Copy link
Member

t3chguy commented May 23, 2022

Fixed by matrix-org/matrix-react-sdk#8590

@t3chguy t3chguy closed this as completed May 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Notifications P1 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Product More input needed from the Product team
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dbkr @Pointedstick @t3chguy