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 699173 - Implementation of Playlist View
Implementation of Playlist View
Status: RESOLVED FIXED
Product: gnome-music
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: 3.12
Assigned To: gnome-music-maint
gnome-music-maint
Depends on:
Blocks: 702866
 
 
Reported: 2013-04-28 19:49 UTC by Eslam Mostafa
Modified: 2014-02-17 11:08 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch for browsing playlists (3.27 KB, patch)
2013-06-16 05:18 UTC, Arnel Borja
none Details | Review
Playlists view demo [DON'T COMMIT] (5.34 KB, patch)
2013-06-16 05:22 UTC, Arnel Borja
none Details | Review
Patch for functions for browsing playlists (3.41 KB, patch)
2013-06-16 13:18 UTC, Arnel Borja
none Details | Review
Playlists view demo [don't commit] (4.53 KB, patch)
2013-06-16 13:23 UTC, Arnel Borja
none Details | Review

Description Eslam Mostafa 2013-04-28 19:49:45 UTC
Playlist view is missing, 
it should contain
1- creating new playlists
2- deleting playlists
3- editing existing playlists

this is the presented UI for the playlistview:
https://github.com/gnome-design-team/gnome-mockups/raw/master/music/music-playlists.png
Comment 1 Arnel Borja 2013-06-16 05:18:54 UTC
Created attachment 246923 [details] [review]
Patch for browsing playlists

Functions for browsing playlists.

Make sure you have grilo with playlists support before testing:
branches['grilo'] = ('git://git.collabora.co.uk/git/user/mbatle/grilo.git', 'playlist-lib')
branches['grilo-plugins'] = ('git://git.collabora.co.uk/git/user/mbatle/grilo-plugins.git', 'playlist-plugin')
Comment 2 Arnel Borja 2013-06-16 05:22:42 UTC
Created attachment 246924 [details] [review]
Playlists view demo [DON'T COMMIT]

Since the patch for the playlists view implementation is not pushed yet, this is what I do to test the new functions.

Selecting a playlist on the left pane will print the song titles in the terminal.

Make sure that "$HOME/.local/share/gnome-music/playlists" have some playlist files. Create this directory if it doesn't exists. You could use totem to create a playlist file (tested with m3u and pls playlists).
Comment 3 Arnel Borja 2013-06-16 05:38:20 UTC
Oh, I'm not sure if the config part is important, I just copied it from grilo playlist examples.
Comment 4 Arnel Borja 2013-06-16 13:18:47 UTC
Created attachment 246946 [details] [review]
Patch for functions for browsing playlists

Remove config. check first if playlist dir exists, creating that at startup if it doesn't. added missing options.
Comment 5 Arnel Borja 2013-06-16 13:23:51 UTC
Created attachment 246948 [details] [review]
Playlists view demo [don't commit]

Removed some trivial things.

Note that currently we don't have a way to handle Load More button on both panes of Playlists view, since View assumes we use grl-tracker-source, which supports queries, which grl-filesystem doesn't support.
Comment 6 Vadim Rutkovsky 2014-02-17 11:08:07 UTC
Initial playlists support was added in https://git.gnome.org/browse/gnome-music/commit/?id=2609785e5074538d1fb2dab0f0d2cf5e5d791103