Review Request 118891: Folder view icon text background

Andrew Lake jamboarder at gmail.com
Mon Jun 23 14:19:41 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118891/
-----------------------------------------------------------

(Updated June 23, 2014, 2:19 p.m.)


Review request for Plasma.


Changes
-------

Use units.smallspacing for text background margins. Don't show text background on popups or widget view. Also decreased opacity to 0.4 to reduce "heavy" feel.  See attached screen capture.


Bugs: 335070
    https://bugs.kde.org/show_bug.cgi?id=335070


Repository: plasma-desktop


Description
-------

Addresses lack of contrast of folderview containment icon text on certain backgrounds: Bug 335070

The color of the text background is just the complement of the icon label text with a 0.6 opacity applied.


Diffs (updated)
-----

  containments/folder/package/contents/ui/ConfigIcons.qml 9f57900 
  containments/folder/package/contents/ui/ItemDelegate.qml 4f95f04 

Diff: https://git.reviewboard.kde.org/r/118891/diff/


Testing
-------


File Attachments (updated)
----------------

with latest changes showing it with selection background
  https://git.reviewboard.kde.org/media/uploaded/files/2014/06/23/58f07e42-08b4-480a-9c05-40195514edbf__icontextbackground2.png


Thanks,

Andrew Lake

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140623/289df8ef/attachment-0001.html>


More information about the Plasma-devel mailing list