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.

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

Notification counters are not aligned #27967

Closed
kongo09 opened this issue Aug 27, 2024 · 2 comments
Closed

Notification counters are not aligned #27967

kongo09 opened this issue Aug 27, 2024 · 2 comments
Assignees
Labels
A-Notifications O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@kongo09
Copy link

kongo09 commented Aug 27, 2024

Steps to reproduce

  1. Observe the notification counter in the panel on the left - it doesn't reflect the sum of notifications in the room list on the right

image

  1. Observe a new message coming in. The counter on the left goes up 2, not 1

image

  1. Read the new message. The counter on the left goes down 1

image

Outcome

What did you expect?

Counter on the left to be consistent with the sum of counters on the right

What happened instead?

Counter on the left feels arbitrary

Discussed here: https://matrix.to/#/!bYSNiRQzSqoaVuYiFK:matrix.org/$veoDwR2f3jTMFL13le_C6Wtt0CxxV_PsaY5-pABzBi8?via=element.io&via=matrix.org&via=one.ems.host

Operating system

Windows 11

Application version

Element version: 1.11.75 Crypto version: Rust SDK 0.7.1 (c8c9d15), Vodozemac 0.6.0

How did you install the app?

element.io

Homeserver

element.io

Will you send logs?

Yes

@dosubot dosubot bot added A-Notifications O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist labels Aug 27, 2024
@daniellekirkwood
Copy link
Contributor

@florianduros , you're maintainer this week? Would you take a look 🙏

@florianduros florianduros self-assigned this Aug 28, 2024
@t3chguy
Copy link
Member

t3chguy commented Aug 28, 2024

Duplicate of #20372

@t3chguy t3chguy marked this as a duplicate of #20372 Aug 28, 2024
@t3chguy t3chguy closed this as not planned Won't fix, can't repro, duplicate, stale Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Notifications O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

No branches or pull requests

5 participants
@kongo09 @t3chguy @florianduros @daniellekirkwood and others