jukebox2 icon indicating copy to clipboard operation
jukebox2 copied to clipboard

#feature - adding a song should be an ajax request

Open deobald opened this issue 14 years ago • 6 comments

Can someone with admin powers label this as a feature?

deobald avatar Mar 31 '12 14:03 deobald

Also, you should be able to add a whole album. We have such features on a branch! MAGIC POWER SAUCE

deobald avatar Mar 31 '12 14:03 deobald

Adding a song = adding to playlist?

If so, this is already the case. If you search for a song and click the title (assuming you're logged in) the request is made in the background with the playlist updating.

cmonty avatar Mar 31 '12 16:03 cmonty

Right now it redirects you to the playlist when you queue a song. So if you want to queue 4 songs you have to queue-back-queue-back-queue-back-queue-back. Working on a fix. :) For the time being, I've added "add album" functionality... on the way!

deobald avatar Mar 31 '12 17:03 deobald

Can you re-open this? Or am I misunderstanding something? I have another patch for a similar issue (queuing albums).

deobald avatar Mar 31 '12 17:03 deobald

You shouldn't be redirected if you're using search. If you're browsing the library, it'll use a standard browser request. I can see how that's annoying...

I think adding a whole album should be a separate issue, not included in with the AJAX requests when browsing.

cmonty avatar Mar 31 '12 17:03 cmonty

Yeah, sorry... was conflating ajax-adds with album-adds because I was lazy. I'll come up with a separate patch for this issue since I didn't finish ajax-adds on that branch on Saturday anyway.

deobald avatar Apr 02 '12 17:04 deobald