Bug 384015 - Improve visibility of notifications icon in system tray
Summary: Improve visibility of notifications icon in system tray
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Theme - Breeze (show other bugs)
Version: 5.16.4
Platform: Arch Linux Linux
: NOR normal
Target Milestone: 1.0
Assignee: visual-design
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-08-25 17:57 UTC by Patrick Silva
Modified: 2019-08-09 13:47 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.62
kde: VisualDesign+


Attachments
screenshot showing grey and almost invisible icon (23.08 KB, image/png)
2017-08-25 17:58 UTC, Patrick Silva
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2017-08-25 17:57:40 UTC
Notifications icon is system tray is almost invisible when I use dark theme.
And it's inconsistent because others icons in system tray are white and notifications icon is grey.
Comment 1 Patrick Silva 2017-08-25 17:58:31 UTC
Created attachment 107516 [details]
screenshot showing grey and almost invisible icon
Comment 2 Kai Uwe Broulik 2017-08-28 09:34:19 UTC
It's gray when there's nothing going on. If there's somehing going on it's white and/or blue, no?
Comment 3 andreas 2017-08-28 09:44:32 UTC
as fare as I know it's light gray when nothing is happend and blue when you get a notification.

I also thought it should be the default gray when nothing going on. cause now there is an opacity.

if someone of the plasma theme say change to default gray (text color) than I can submitt the patch.
Comment 4 Nate Graham 2017-10-30 20:47:10 UTC
Go ahead and submit the patch, Andreas! The worst that can happen is that it'll be rejected. :)
Comment 5 Nate Graham 2019-08-09 13:43:49 UTC
Git commit 134b2ac331d373d06d662f88f5c29dc09c8ebd0d by Nate Graham, on behalf of Noah Davis.
Committed on 09/08/2019 at 13:43.
Pushed by ngraham into branch 'master'.

Make notification icons look like bells

Summary: BUG: 384015

Test Plan:
See D23018

{F7178298, size=full}

Reviewers: #vdg, #plasma, broulik, ngraham

Reviewed By: #vdg, #plasma, broulik, ngraham

Subscribers: ngraham, broulik, kde-frameworks-devel

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D23019

M  +42   -272  src/desktoptheme/breeze/icons/notification.svg

https://commits.kde.org/plasma-framework/134b2ac331d373d06d662f88f5c29dc09c8ebd0d