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 699592 - Artist list - highlight the selected artist
Artist list - highlight the selected artist
Status: RESOLVED FIXED
Product: gnome-music
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-music-maint
gnome-music-maint
Depends on:
Blocks:
 
 
Reported: 2013-05-03 14:56 UTC by Allan Day
Modified: 2013-05-11 20:36 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Changed the background color of the artists column in artists view. (794 bytes, patch)
2013-05-08 22:00 UTC, Sai Suman Prayaga
rejected Details | Review
Changed the background and highlight through css (2.13 KB, patch)
2013-05-11 15:18 UTC, Sai Suman Prayaga
accepted-commit_now Details | Review

Description Allan Day 2013-05-03 14:56:28 UTC
The artist list doesn't indicate which artist is currently selected. The selected row should have a darker color, and the color of the text in the selected row should be inverted (so that it is the same as the background).

See: https://raw.github.com/gnome-design-team/gnome-mockups/master/music/music-artists.png
Comment 1 Sai Suman Prayaga 2013-05-08 22:00:25 UTC
Created attachment 243629 [details] [review]
Changed the background color of  the artists column in artists  view.
Comment 2 Sai Suman Prayaga 2013-05-08 22:05:28 UTC
Review of attachment 243629 [details] [review]:

Sorry, Wrong bug patch.
Comment 3 Seif Lotfy 2013-05-10 13:47:22 UTC
Review of attachment 243629 [details] [review]:

Wrong patch
Comment 4 Sai Suman Prayaga 2013-05-11 15:18:20 UTC
Created attachment 243851 [details] [review]
Changed the background and highlight through css
Comment 5 Vadim Rutkovsky 2013-05-11 16:01:52 UTC
Review of attachment 243851 [details] [review]:

Nice job, approved