D21027: [Notes] Fix icons being almost invisible with light backgrounds

Nathaniel Graham noreply at phabricator.kde.org
Sun May 5 19:49:24 BST 2019


ngraham added a comment.


  In D21027#461027 <https://phabricator.kde.org/D21027#461027>, @filipf wrote:
  
  > I get what you're saying but that's just how the applet is designed. It's preconfigured SVG backgrounds (you can find them in `/usr/share/plasma/desktoptheme/default/widgets/notes.svgz`). I'm not really introducing any hardcoding that isn't already there. See the hardcoding of text color:
  >
  >   style: PlasmaStyle.TextAreaStyle {
  >       //this is deliberately _NOT_ the theme color as we are over a known bright background
  >       //an unknown colour over a known colour is a bad move as you end up with white on yellow
  >       textColor: (plasmoid.configuration.color === "black" || plasmoid.configuration.color === "translucent-light") ? "#dfdfdf" : "#202020"
  >   }
  >   
  >
  > So one thing is if we'd want to rewrite the whole thing to be a rectangle and then you could select a huge amount of colors as the background I guess, as well as have a spinbox for opacity. Also tagging @broulik as he might know better why it's not like that in the first place.
  
  
  Gotcha. At least use a function instead of duplicating the same code in each button.
  
  > The other thing about components is that Plasma Components doesn't support `icon.color`. I'd have kept using it, but PC2 spews out an error, while with PC3 it didn't complain but didn't work anyway.
  
  OK, that makes sense. Can you make it a bit more clear in the Description?

REPOSITORY
  R114 Plasma Addons

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

To: filipf, #plasma, #vdg, ngraham
Cc: broulik, davidedmundson, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190505/c3b31476/attachment.html>


More information about the Plasma-devel mailing list