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 699824 - No empty state when you have no songs in the Music folder
No empty state when you have no songs in the Music folder
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-07 11:46 UTC by Fabiana Simões
Modified: 2013-05-13 06:58 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Artist View - Empty State (11.57 KB, image/png)
2013-05-07 11:46 UTC, Fabiana Simões
  Details
Artist View - Empty State (11.83 KB, image/png)
2013-05-07 11:48 UTC, Fabiana Simões
  Details
Proposed ui for No Music Found (4.85 KB, patch)
2013-05-09 23:19 UTC, Shivani Poddar
needs-work Details | Review
Solves the bug (5.46 KB, patch)
2013-05-12 21:13 UTC, Shivani Poddar
needs-work Details | Review
Fixes the Bug (5.26 KB, patch)
2013-05-13 06:49 UTC, Shivani Poddar
committed Details | Review

Description Fabiana Simões 2013-05-07 11:46:23 UTC
Created attachment 243473 [details]
Artist View - Empty State

When you have no songs in the Music folder, when you open gnome-music, it just shows a blank window, with some controls in the bottom (see attachment).

I was confused by this, thinking some error had occurred, until Andreas pointed me I had no songs in the Music folder.

I guess the application should have some empty state as, for example, Documents have: https://github.com/gnome-design-team/gnome-mockups/raw/master/documents/documents-empty.png
Comment 1 Fabiana Simões 2013-05-07 11:48:47 UTC
Created attachment 243476 [details]
Artist View - Empty State

Also, the Artist View, when you have no songs in the Music folder, looks even more error-state-ish (The others look like the screenshot for the Albums view)
Comment 2 Fabiana Simões 2013-05-07 11:50:36 UTC
Oh, Attachment 243473 [details] is Albums View not Artist View.
Comment 3 Allan Day 2013-05-08 08:50:34 UTC
Visually, I would expect this to be in the same style as the Documents empty state:

https://raw.github.com/gnome-design-team/gnome-mockups/master/documents/documents-empty.png

We need to make sure that the text matches the possibilities for adding music to your system, of course.

I would also remove the Albums / Artists / Songs / Playlists toggles from the toolbar when the application is empty.
Comment 4 Shivani Poddar 2013-05-09 23:19:09 UTC
Created attachment 243749 [details] [review]
Proposed ui for No Music Found

Glade file to go with the mockup.
Also TODO in the ui the link to "system settings" as described here -> https://raw.github.com/gnome-design-team/gnome-mockups/master/documents/documents-empty.png
Comment 5 Seif Lotfy 2013-05-10 12:26:11 UTC
Thanks Shivani :D
Can you bind it with the code :)
Comment 6 Vadim Rutkovsky 2013-05-11 16:46:41 UTC
Review of attachment 243749 [details] [review]:

This should be loaded initially at artists view and then replaced by existing UI when any artist is found
Comment 7 Shivani Poddar 2013-05-12 21:13:05 UTC
Created attachment 243953 [details] [review]
Solves the bug
Comment 8 Vadim Rutkovsky 2013-05-12 22:24:58 UTC
Review of attachment 243953 [details] [review]:

Good job! However the icon seems misaligned with the text - label should be centered with the icon horizontally
Comment 9 Shivani Poddar 2013-05-13 06:49:23 UTC
Created attachment 243966 [details] [review]
Fixes the Bug

Updated patch.
Comment 10 Shivani Poddar 2013-05-13 06:50:02 UTC
(In reply to comment #8)
> Review of attachment 243953 [details] [review]:
> 
> Good job! However the icon seems misaligned with the text - label should be
> centered with the icon horizontally

Have attached the correct patch now, by mistake put the wrong glade file in the earlier patch :)
Comment 11 Vadim Rutkovsky 2013-05-13 06:55:56 UTC
Review of attachment 243966 [details] [review]:

Thanks, pushed to master
Comment 12 Shivani Poddar 2013-05-13 06:58:09 UTC
(In reply to comment #11)
> Review of attachment 243966 [details] [review]:
> 
> Thanks, pushed to master

Thankyou :D