Skip to content

Conversation

@fredcw
Copy link
Contributor

@fredcw fredcw commented Nov 21, 2025

Depends on #12569

@fredcw fredcw changed the title window-list applet: Add notification badges [WIP]window-list applet: Add notification badges Nov 21, 2025
@fredcw fredcw changed the title [WIP]window-list applet: Add notification badges window-list applet: Add notification badges Nov 21, 2025
@fredcw fredcw changed the title window-list applet: Add notification badges [WIP] window-list applet: Add notification badges Nov 28, 2025
@fredcw fredcw changed the title [WIP] window-list applet: Add notification badges window-list applet: Add notification badges Nov 30, 2025
@fredcw fredcw force-pushed the windowlistnotifications branch from a32fabd to 44a520f Compare December 3, 2025 09:38
@clefebvre clefebvre merged commit 7abbbe8 into linuxmint:master Dec 10, 2025
3 checks passed
@fredcw fredcw deleted the windowlistnotifications branch December 10, 2025 16:42
@mtwebster
Copy link
Member

I don't know about this:
image

I ejected a usb stick, and this is the result of the 'it is safe to remove your device' notification.

For the GWL it works out when using app-grouping, but here, I'm not so sure, since we'll have this with any app that supports multiple windows.

@fredcw
Copy link
Contributor Author

fredcw commented Dec 16, 2025

@mtwebster I'm not sure there's anything we can do about this. notificationDaemon.js only creates new notification sources based on pid. I'm guessing there's not enough info in a notification to distinguish between different windows of the same app.

@fredcw
Copy link
Contributor Author

fredcw commented Dec 20, 2025

@mtwebster It appears there's no reliable or consistent way to associate a notification with any particular window. The only other options I can think of: only put the badge on the most recently active window (though this would probably be misleading more often than not and wouldn't account for multiple monitors and workspaces) or only put a badge on the leftmost or rightmost icon in the panel but this too may be potentially confusing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants