ngraham accepted this revision.
ngraham added a comment.

  Never mind, you did change it to `notification-inactive` in the other patch. 
So this is good to go IMO.
  
  As for landing it, we have two options to make sure that rolling release 
distro users don't see their notification icon change to the one with the red 
line through it:
  
  1. Land this patch for 5.63 (the Frameworks release aligned with Plasma 5.17)
  2. Change the inactive applet to use the `notification-inactive icon` in 
another patch and land that on the stable branch, then land this

REPOSITORY
  R242 Plasma Framework (Library)

REVISION DETAIL
  https://phabricator.kde.org/D23019

To: ndavis, #vdg, #plasma, broulik, ngraham
Cc: ngraham, broulik, kde-frameworks-devel, LeGast00n, michaelh, bruns

Reply via email to