D17975: [Notifications] Add padding to notifications

[David Edmundson]

davidedmundson added a comment.


  > Is there a way to give the icon itself padding? I've been using anchors
  
  Yes, but if you're padding the icon by moving it then you need to set the window height to
  
  Math.max(sizeOfIcon, sizeOfText)
  
  needs to be
  
  Math.max(sizeOfIcon + 2*padding, sizeOfText)
  
  in order to keep even padding.
  
  ----
  
  > How do we get the icon size to increase in proportion to increasing font size
  
  See units.cpp in plasma-framework. It's complex. I don't think that's a relevant topic you need to go down

REPOSITORY
  R120 Plasma Workspace

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

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