After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 558656 - Recommendations Pane: Artist color shows up incorrectly with DarkRoom theme
Recommendations Pane: Artist color shows up incorrectly with DarkRoom theme
Status: RESOLVED FIXED
Product: banshee
Classification: Other
Component: Other Extensions
git master
Other Linux
: Normal trivial
: 1.x
Assigned To: Banshee Maintainers
Banshee Maintainers
: 561092 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2008-10-31 12:44 UTC by Andrew Conkling
Modified: 2009-05-13 01:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Set foreground to text colour on misbehaving widgets (2.13 KB, patch)
2008-12-28 17:57 UTC, Chris Jones
reviewed Details | Review
Set foreground to text colour on misbehaving widgets with correct coding style (2.14 KB, patch)
2009-02-08 16:19 UTC, Chris Jones
committed Details | Review

Description Andrew Conkling 2008-10-31 12:44:19 UTC
Filed downstream: https://bugs.launchpad.net/ubuntu/+source/banshee/+bug/291322.

The Recommendations pane artist text shows up incorrectly. See the screenshot: http://launchpadlibrarian.net/19099766/Banshee.png

Not sure if there's a Banshee bug here (or just a theme one), but figured I'd file it here in case anything can be done. (I know how much you love working around GTK+ theme bugs.)

(Of course, since it's Ubuntu, the theme doesn't seem to be available on its own, but here's the best I can do quickly: http://packages.ubuntu.com/intrepid/human-theme.)
Comment 1 Bertrand Lorentz 2008-11-16 19:47:10 UTC
*** Bug 561092 has been marked as a duplicate of this bug. ***
Comment 2 Chris Jones 2008-12-28 17:57:47 UTC
Created attachment 125441 [details] [review]
Set foreground to text colour on misbehaving widgets
Comment 3 Chris Jones 2008-12-28 18:02:07 UTC
Comment on attachment 125441 [details] [review]
Set foreground to text colour on misbehaving widgets

he problem occurs with themes that set different colours for foreground and text. Most themes set foreground and text the same and background and base the same. DarkRoom sets different colours for each of them.

The foreground colour contrasts with the background and text contrasts with base.

Banshee sets the widgets background colour to the theme's base colour but doesn't set the foreground colour to the theme's text colour.
Comment 4 Gabriel Burt 2009-01-13 22:31:50 UTC
Hey Chris, thanks for the patch.  Please put spaces between method names and arg lists eg likeThis ().  See HACKING for our full style guide.
Comment 5 Chris Jones 2009-02-08 16:19:56 UTC
Created attachment 128234 [details] [review]
Set foreground to text colour on misbehaving widgets with correct coding style
Comment 6 Gabriel Burt 2009-02-20 21:29:30 UTC
Thanks Chris, committed.
Comment 7 Alexander Kojevnikov 2009-05-13 01:47:47 UTC
*** Bug 582377 has been marked as a duplicate of this bug. ***