Add possibility to remove from library

detcherry / phonoblaster

Phonoblaster application (App Engine)

4 stars 2 forks source link

Add possibility to remove from library #15

Closed detcherry closed 12 years ago

detcherry commented 12 years ago

It should be possible to remove a track from your entire library. This was possible a few weeks ago. This possibility should come back

Boussadia commented 12 years ago

@detcherry : question regarding the UX, for the moment here is what I implemented : if the user requires to delete a track that is in the buffer (not necessarily curently broadcasting) the action is rejected. ok, not ok?

detcherry commented 12 years ago

I think this is a good idea to reject an action "delete a track from my library" if the track is currently in the buffer To do it properly, it should be implemented on both client and server side.

On the client side: If I try to click on the cross. I have an alert popup saying "Delete this track from your buffer first."
On the server side: the controller checks if the track is not in the buffer. If no, response: true. If yes, response: false

Boussadia commented 12 years ago