Restructure project - Githubissues

Glavin001 / atom-playlist

(This package is currently not maintained) Queue and play music directly from Atom. (deep in development)

6 stars 0 forks source link

Restructure project #12

Closed Glavin001 closed 10 years ago

Glavin001 commented 10 years ago

See https://github.com/Glavin001/atom-tunes

Glavin001 commented 10 years ago

Keep the existing atom playlist's Playlist view. @dreid93 made it pretty neat lookin'.

Glavin001 commented 10 years ago

@dreid93 what about renaming to just Playlist or even Tunes. I noticed that neither are taken, and we do not need the atom- prefix ;). Will have to republish though and existing users will not get update. We should make a special mention of that in the README and publish before changing name.

Glavin001 commented 10 years ago

[x] Also change the GitHub project title to something like "Ultimate Music player for Atom!"

ddaws commented 10 years ago

On APM it called just playlist. It would be nice to capitalize.

Glavin001 commented 10 years ago

Oh good, I thought it was atom-playlist. Great you already have the playlist name.

ddaws commented 10 years ago

For future reference : I support this restructuring though I would like to retain the use of the QueueView over atom-tune's PlaylistView. I am not opposed to renaming QueueView to PlaylistView though I think it may be more confusing because there will be a playlist.coffee and a playlist_view.coffee file. The solution may be to rename playlist.coffee to index.coffee.

Glavin001 commented 10 years ago

Makes sense. PlaylistView should be QueueView and Playlist model should be Queue (model). :+1: