Bug 429862 - "No unread notification" when there are notifications in the tray
Summary: "No unread notification" when there are notifications in the tray
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Notifications (other bugs)
Version First Reported In: 5.20.3
Platform: Neon Linux
: NOR normal
Target Milestone: 1.0
Assignee: Kai Uwe Broulik
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-11-30 18:07 UTC by Riccardo Robecchi
Modified: 2023-04-29 08:57 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Screenshot of the notification tray (40.24 KB, image/png)
2020-11-30 18:07 UTC, Riccardo Robecchi
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Riccardo Robecchi 2020-11-30 18:07:09 UTC
Created attachment 133755 [details]
Screenshot of the notification tray

SUMMARY
Sometimes the notification tray shows "no unread notification" even if there are, in fact, notifications there.

STEPS TO REPRODUCE
1. Receive a notification
2. let it time out so it goes in the tray
3. open the tray

OBSERVED RESULT
The notification is shown along with the text "no unread notification".

EXPECTED RESULT
The notification is shown without any further text.

SOFTWARE/OS VERSIONS
Linux: KDE neon Focal
KDE Plasma Version: 5.20.3
KDE Frameworks Version: 5.76.0
Qt Version: 5.15.1

ADDITIONAL INFORMATION
Comment 1 Nate Graham 2020-11-30 22:33:54 UTC
That's weird. Cannot reproduce.
Comment 2 Kai Uwe Broulik 2021-01-24 09:56:59 UTC
I've sometimes seen this happen, or at least the "clear history" was grayed out indicating that it didn't think there were unread notifications. Alas, I haven't managed to reproduce it reliably.
Comment 3 Kai Uwe Broulik 2021-01-26 21:26:35 UTC
Git commit 67f9d5031bc388cd627e49ece63fc672d814a17d by Kai Uwe Broulik.
Committed on 26/01/2021 at 21:26.
Pushed by broulik into branch 'master'.

[Notifications] Re-evaluate unread count when a notification is "read"

This is when a notification was hovered and thus "acknowledged", which was
added after the original `updateCount` implementation.

M  +2    -1    libnotificationmanager/notifications.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/67f9d5031bc388cd627e49ece63fc672d814a17d
Comment 4 Kai Uwe Broulik 2021-01-26 21:27:01 UTC
Git commit f39049578967c410c6c23c5e91d33e2e3a61a196 by Kai Uwe Broulik.
Committed on 26/01/2021 at 21:26.
Pushed by broulik into branch 'Plasma/5.21'.

[Notifications] Re-evaluate unread count when a notification is "read"

This is when a notification was hovered and thus "acknowledged", which was
added after the original `updateCount` implementation.


(cherry picked from commit 67f9d5031bc388cd627e49ece63fc672d814a17d)

M  +2    -1    libnotificationmanager/notifications.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/f39049578967c410c6c23c5e91d33e2e3a61a196
Comment 5 Riccardo Robecchi 2021-01-27 23:10:38 UTC
(In reply to Kai Uwe Broulik from comment #3)
> This is when a notification was hovered and thus "acknowledged", which was
> added after the original `updateCount` implementation.

I'm not sure this is the case: it just happened to me again with a KDE Connect notification which I haven't hovered as I wasn't at the PC. We'll see once I'll be able to test this patch though.
Comment 6 Kai Uwe Broulik 2021-03-14 19:02:50 UTC
Git commit d840c5dca2090c90d7b7d5633d80f85d9f8758db by Kai Uwe Broulik.
Committed on 14/03/2021 at 19:02.
Pushed by broulik into branch 'master'.

[Notifications] Emit dataChanged in setData call

Ensures the role is properly signalled to have changed.

M  +7    -2    libnotificationmanager/abstractnotificationsmodel.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/d840c5dca2090c90d7b7d5633d80f85d9f8758db
Comment 7 Kai Uwe Broulik 2021-03-14 19:06:09 UTC
Git commit 73c104ac393583599fc7fecc3a8934ca392cb35a by Kai Uwe Broulik.
Committed on 14/03/2021 at 19:06.
Pushed by broulik into branch 'Plasma/5.21'.

[Notifications] Emit dataChanged in setData call

Ensures the role is properly signalled to have changed.


(cherry picked from commit d840c5dca2090c90d7b7d5633d80f85d9f8758db)

M  +7    -2    libnotificationmanager/abstractnotificationsmodel.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/73c104ac393583599fc7fecc3a8934ca392cb35a
Comment 8 Riccardo Robecchi 2021-09-27 11:10:43 UTC
This still happens all the time with Plasma 5.22.5 and Frameworks 5.86.0.
Comment 9 Nate Graham 2021-09-27 15:12:01 UTC
Just with Spectacle's notifications, or with any others too?
Comment 10 Riccardo Robecchi 2021-09-27 15:12:57 UTC
I've seen it with KDE Connect, Thunderbird, Slack and Spectacle notifications among others.
Comment 11 Nate Graham 2023-04-28 04:53:37 UTC
Is it still happening in Plasma 5.27, Riccardo?
Comment 12 Riccardo Robecchi 2023-04-29 08:57:05 UTC
No, I haven't experienced it in a while. I'm closing this down as fixed. Thanks to all people involved!