search

mixu / nplay

Console-based mp3 player with Winamp key bindings and jump-to-file
http://mixu.net/nplay/
43 stars 7 forks source link

this package is awesome #6

Open Streemo opened 9 years ago

Streemo commented 9 years ago

Thanks for this awesome package. It has officially replaced iTunes for me. Somehow, iTunes manages to be 10 times more complicated than this command line tool. I have just a few things to mention:

Search:

Files are often stored like this:

/music/artist/obscureSongNameOfArtistThatIForgotButIKnowTheNameOfTheArtist.mp3

I hope the need for path search now makes sense. Basically, no one remembers song names, but everyone remembers the artist. To get around this, I am now appending artistName_ before all of my songs (programmatically since my artistName === my parent folder name, so no worries)

Traversal:

Should be able to traverse faster (using mac) than just holding the up and down arrow keys. Options include accelerating (with plateau) scroll speed, or some keys which skip N songs in the list at a time.

mixu commented 9 years ago

:+1: This is actually the one project I wish was more popular :) even though it wasn't particularly complicated to write, I really wish more media players had jump to file (with some smarts, e.g. weighed by plays and ratings). In particular, I'd like to do an Android port but haven't quite had the time to do any serious Android programming since currently - haven't found anything particularly great so I've just been surviving with https://play.google.com/store/apps/details?id=com.myskyspark.music&hl=en

Adding path support sounds good, I think it would be a matter of patching https://github.com/mixu/nplay/blob/master/lib/playlist.js#L104 to use path instead of name and then changing https://github.com/mixu/nplay/blob/master/lib/templates/jump.js (called from https://github.com/mixu/nplay/blob/master/lib/modes/jump.js#L28 ) to show paths. Would be great if typing somedir/ or /somedir would be recognized correctly as a query for a path segment and treated specially. I'll take a look the next time I'm working on nplay, or feel free to take a look if you feel like it (keeping this issue open as a TODO).

For traversal, there are actually a couple of shortcuts, I'll update the docs:

I feel like the navigation would be a bit better if the currently highlighted line was a bit more obvious. I've been thinking about changing the formatting to be inverted, e.g. black text on yellow background for the current song.

Let me know if you have additional ideas!

Streemo commented 9 years ago

also, the whole row should be highlighted, not just the name of the song. It will then be easier to see the play count, last played, etc. without having to manually match them up in your head.

yeah I'll take a look at the source when I have some more time! I too agree, this needs more attention. You've done some useful work here, even if it was simple! On Oct 27, 2015 12:36 AM, "Mikito Takada" notifications@github.com wrote:

[image: :+1:] This is actually the one project I wish was more popular :) even though it wasn't particularly complicated to write, I really wish more media players had jump to file (with some smarts, e.g. weighed by plays and ratings). In particular, I'd like to do an Android port but haven't quite had the time to do any serious Android programming since currently - haven't found anything particularly great so I've just been surviving with https://play.google.com/store/apps/details?id=com.myskyspark.music&hl=en

Adding path support sounds good, I think it would be a matter of patching https://github.com/mixu/nplay/blob/master/lib/playlist.js#L104 to use path instead of name and then changing https://github.com/mixu/nplay/blob/master/lib/templates/jump.js (called from https://github.com/mixu/nplay/blob/master/lib/modes/jump.js#L28 ) to show paths. Would be great if typing somedir/ or /somedir would be recognized correctly as a query for a path segment and treated specially. I'll take a look the next time I'm working on nplay, or feel free to take a look if you feel like it (keeping this issue open as a TODO).

For traversal, there are actually a couple of shortcuts, I'll update the docs:

  • Page up (fn + up arrow on a Macbook) / Page down (fn + down) jump half a screen's worth (should it be a full screen??)
  • Esc returns back to the currently playing song
  • Home and End (fn + left / right) go to the top and bottom of the playlist

I feel like the navigation would be a bit better if the currently highlighted line was a bit more obvious. I've been thinking about changing the formatting to be inverted, e.g. black text on yellow background for the current song.

Let me know if you have additional ideas!

— Reply to this email directly or view it on GitHub https://github.com/mixu/nplay/issues/6#issuecomment-151402431.