Version: (using Devel) Installed from: Compiled sources OS: Linux The device-notifier applet popup title text, "Recently Plugged Devices", is hard coded white. Therefore it is difficult to read with plasma themes that have a lighter background. Steps to Reproduce: 1. Load the Aya Plasma theme 2. Select the default colorscheme in the Colors module of system settings. 3. Click on Device Notifier and notice text remains white. Does not correct itself on restart. A review of the source shows that the label is CSS hard-coded white.
This was fixed by duffeck in commit http://websvn.kde.org/?view=rev&revision=792851
*** Bug has been marked as fixed ***.
Created attachment 25395 [details] Attachment That Shows Relapse In Bug This Bug Has Regressed. Requesting Bug Be Reopened
FYI: Using KDE4Daily Rev 820990
Adding Self To CC, Also Additional Information: Plasma Theme: Aya Colour Theme: Oxygen Additional Testing By Placing The Another Device Notifer Applet On The Desktop Has Revealed That It Then Sets The Text Colour Correctly, Until The Plasma Theme Is Changed. When The Plasma Theme Is Changed The Text Colour DOES NOT Update Itself. This Is A Different Bug, Should I Create A New Bug?
Created attachment 25396 [details] Believed Fix, Altered Revision 820974 Patch, But Not In Patch Format, Complete File Can't Compile / Test But This Should Work As Far As I Know. What I Did: Created Slot At Bottom Of File, Just Above #include "devicenotifier.moc" Called updateColors() Connected That At The Bottom Of init() If You Can't See The Differences. If It's Fine, Please Commit, As I Don't Have A SVN Account
Created attachment 25397 [details] sourtooth's changes in proper patch format Turned sourtooth's proposed changes into a patch for easier review. Not able to test either, currently on the road, sorry. original description: "Believed Fix, Altered Revision 820974 Patch, But Not In Patch Format, Complete File Can't Compile / Test But This Should Work As Far As I Know. What I Did: Created Slot At Bottom Of File, Just Above #include "devicenotifier.moc" Called updateColors() Connected That At The Bottom Of init() If You Can't See The Differences. If It's Fine, Please Commit, As I Don't Have A SVN Account "
Created attachment 25413 [details] Proper patch This is the proper patch for allowing change of theme
SVN commit 821474 by annma: let the applet adapt to theme change - please guys test and comment, thanks! CCBUG=160063 M +12 -4 devicenotifier.cpp M +2 -0 devicenotifier.h WebSVN link: http://websvn.kde.org/?view=rev&revision=821474