DIYgod / APlayer

:lollipop: Wow, such a beautiful HTML5 music player
http://aplayer.js.org
MIT License
7.31k stars 1.03k forks source link

Can't find the delete action in the playlist, consider any updates ? #198

Open farwish opened 6 years ago

SevenOutman commented 6 years ago

@farwish You can manipulate your playlist with apis as a developer. For your visitors, because we are focusing on a player rather than a music service - e.g. search by title, so there's no adding songs feature. And that's why removing songs does not make much sense too.

farwish commented 6 years ago

I can not completely agree with it. Adding songs feature is different from removing feature, because 'adding' is not expected on player directly, it is right, 'adding' action often in other place where new songs searched, and event logic developed by developer using addAudio; but nearly all normal user want to 'remove' song on playlist directory, it is a basic functional action like 'repeat' and 'shuffle'.

I think you knows what means i say, playlist missed an section like this <span class="aplayer-list-remove">x</span> after aplayer-list-author https://github.com/MoePlayer/APlayer/blob/master/src/js/player.js#L461

Maybe it also can be optional, if developer want to show it, pass this option { enableRemoveControls: true}

DIYgod commented 6 years ago

At first, APlayer is designed as a simple web audio player which is used for play my favorite music on my blog, I never consider more complicated situations before.

Thanks for your suggest and I'll focus on functionality in the next time.

SevenOutman commented 6 years ago

@DIYgod 's initial goal was what I meant - I understand normal users may want to 'remove' songs, but APlayer wasn't designed for visitors but site owner themselves to decide what songs are there to play. Anyway still nice idea.

farwish commented 6 years ago

Now i understand, thanks for your work on aplayer.

SevenOutman commented 6 years ago

@farwish If you are using APlayer with Vue, vue-aplayer can be a good choice for you.