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 760170 - Songs view - odd check box style and position
Songs view - odd check box style and position
Status: RESOLVED FIXED
Product: gnome-music
Classification: Applications
Component: general
3.18.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-music-maint
gnome-music-maint
Depends on:
Blocks:
 
 
Reported: 2016-01-05 17:43 UTC by Allan Day
Modified: 2016-02-02 12:18 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Checkbutton proposed fix (2.70 KB, patch)
2016-01-27 18:16 UTC, Saiful B. Khan
committed Details | Review
Fixed Albums view (1.13 KB, patch)
2016-02-01 18:51 UTC, Saiful B. Khan
committed Details | Review

Description Allan Day 2016-01-05 17:43:06 UTC
The check boxes that are shown in the songs view (for selection mode) don't look right. They are very small and have an unnecessary border around them.

They look like the overlaid check boxes that are used in grid views, that have been scaled down. This style shouldn't be used in lists - instead, standard checkboxes should be used.

Also, the checkboxes have a non-standard position in the list: they should be placed at the start of the row before the track title, not after it.
Comment 1 Saiful B. Khan 2016-01-20 17:48:45 UTC
(In reply to Allan Day from comment #0)
> The check boxes that are shown in the songs view (for selection mode) don't
> look right. They are very small and have an unnecessary border around them.
> 
> They look like the overlaid check boxes that are used in grid views, that
> have been scaled down. This style shouldn't be used in lists - instead,
> standard checkboxes should be used.
> 
> Also, the checkboxes have a non-standard position in the list: they should
> be placed at the start of the row before the track title, not after it.

I think I corrected the check buttons appearing to be wierd.
I still have to reposition them to start of the row but please do look at these snapshots and tell me whether these will do:
http://postimg.org/image/3zryxbg4x/

In case you are wondering about why they appear to be invisible, well thats the case for the check buttons even in Artist selection mode
http://postimg.org/image/noiafpc8b/
The above snapshot was from the jhbuil(t) gnome-music, whereas the actual Gnome Music installed application shows them perfectly:
http://postimg.org/image/u32in232n/

Maybe its another different bug or a missing dependency on my end. Please let me know what you think.
Comment 2 Saiful B. Khan 2016-01-27 18:16:32 UTC
Created attachment 319870 [details] [review]
Checkbutton proposed fix
Comment 3 Allan Day 2016-02-01 15:36:34 UTC
(In reply to Saiful B. Khan from comment #2)
> Created attachment 319870 [details] [review] [review]
> Checkbutton proposed fix

Looks good to me. One thing I noticed is that the checkboxes for selection mode when viewing a single album also need updating. It would be nice if those could be fixed too.
Comment 4 Saiful B. Khan 2016-02-01 18:51:47 UTC
Created attachment 320211 [details] [review]
Fixed Albums view
Comment 5 Felipe Borges 2016-02-02 08:30:06 UTC
Review of attachment 319870 [details] [review]:

Allan, do we want to move the checkbuttons from the right side of the Song name to the left?
Comment 6 Felipe Borges 2016-02-02 08:36:36 UTC
Review of attachment 320211 [details] [review]:

Sure.
Comment 7 Felipe Borges 2016-02-02 08:48:39 UTC
Review of attachment 319870 [details] [review]:

(In reply to Felipe Borges from comment #5)
> Review of attachment 319870 [details] [review] [review]:
> 
> Allan, do we want to move the checkbuttons from the right side of the Song
> name to the left?

Saiful told me on IRC that he had previously discussed that with Allan Day.
Comment 8 Allan Day 2016-02-02 12:18:13 UTC
(In reply to Felipe Borges from comment #7)
> Review of attachment 319870 [details] [review] [review]:
> 
> (In reply to Felipe Borges from comment #5)
> > Review of attachment 319870 [details] [review] [review] [review]:
> > 
> > Allan, do we want to move the checkbuttons from the right side of the Song
> > name to the left?
> 
> Saiful told me on IRC that he had previously discussed that with Allan Day.

That's right! Thanks for the fix, Saiful!