Bug 203071 - Black box around icon text looks out of place
Summary: Black box around icon text looks out of place
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Unmaintained
Component: widget-folderview (other bugs)
Version First Reported In: unspecified
Platform: openSUSE Linux
: NOR wishlist
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-08-08 12:39 UTC by Ivo Anjo
Modified: 2009-08-27 10:30 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Image showing problem and some suggestions for alternatives (56.75 KB, image/png)
2009-08-08 12:40 UTC, Ivo Anjo
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ivo Anjo 2009-08-08 12:39:59 UTC
Version:            (using KDE 4.3.0)
OS:                Linux
Installed from:    SuSE RPMs

When you select an icon in the folderview widget in 4.3, besides the smooth rectangle around the entire icon, an ugly black box is shown around the icon text.

This effect feels very out of place.

The attached image shows the problem and has some "suggestions" of solutions, based on other ways currently used on plasma and dolphin.
Comment 1 Ivo Anjo 2009-08-08 12:40:39 UTC
Created attachment 35989 [details]
Image showing problem and some suggestions for alternatives
Comment 2 Bertel King, Jr. 2009-08-09 04:03:51 UTC
Any of the suggested solutions would be better than how the widget currently works. When I first saw the black rectangle, I thought the widget was glitching. This is a bad first impression to a widget that has become increasingly impressive.
Comment 3 Fredrik Höglund 2009-08-27 01:47:31 UTC
SVN commit 1016088 by fredrik:

Replace the black focus indicator with a new design suggested by pinheiro.

This element is currently hardcoded but might be made themeable
in the future.

BUG: 203071


 M  +53 -16    style.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1016088
Comment 4 Ivo Anjo 2009-08-27 10:30:37 UTC
Thanks a lot :)

Could you backport it to branch? I would hate having to wait 5 or 6 months to see this fixed.