j-holub / Node-MPV

A NodeJs Module for MPV Player
https://www.npmjs.com/package/node-mpv
MIT License
117 stars 73 forks source link

loop-playlist #75

Closed jordicasals closed 4 years ago

jordicasals commented 4 years ago

Hi,

would it be possible to add the "loop-playlist" function?

Thank you very much!

j-holub commented 4 years ago

Hey :)

yes it's possible and not much work at all actually, thank you for the suggestion. I will add it asap.

In the meantime you can use player.setProperty('loop-playlist', 'inf') to achieve the same effect. Or replace inf with an integer, if you want to have the playlist looped only a certain number of times. Calling player.setProperty('loop-playlist', 'no') will stop looping the playlist.

Cheers!

jordicasals commented 4 years ago

Thank you very much!

I did not remember I was able to do it using the setProperty method. This is perfect

El dl., 20 abr. 2020, 16.51, Jan Holub notifications@github.com va escriure:

Hey :)

yes it's possible and not much work at all actually, thank you for the suggestion. I will add it asap.

In the meantime you can use player.getProperty('loop-playlist', 'inf') to achieve the same effect. Or replace inf with an integer, if you want to have the playlist looped only a certain number of times. Calling `player.getProperty('loop-playlist', 'no') will stop looping the playlist.

Cheers!

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/j-holub/Node-MPV/issues/75#issuecomment-616604547, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAVCNKOPMEDWJK4U2BMGGBTRNROP5ANCNFSM4MMOLQUA .

j-holub commented 4 years ago

I added a loopPlaylist() and a clearLoopPlaylist() method to Version 1 of the package and published it. The API is fitting to how the Methods in Version 1 look, however I'm not really happy about it yet, so I have not yet implemented anything for Version 2 yet. If you happen to use Version 2 (which you should, it's much better than Version 1), please continue using the setProperty()w way in the meantime.

jordicasals commented 4 years ago

Hi,

I've implemented the software with the setProperty method and it works perfectly. Thank you.

Anyway, as I am using the version 2, I will wait for future updates.

Best regards,

El dt., 21 abr. 2020, 21.33, Jan Holub notifications@github.com va escriure:

I added a loopPlaylist() and a clearLoopPlaylist() method to Version 1 of the package and published it. The API is fitting to how the Methods in Version 1 look, however I'm not really happy about it yet, so I have not yet implemented anything for Version 2 yet. If you happen to use Version 2 (which you should, it's much better than Version 1), please continue using the setProperty()w way in the meantime.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/j-holub/Node-MPV/issues/75#issuecomment-617368949, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAVCNKI5SDLHCQW5Y5DD2IDRNXYIPANCNFSM4MMOLQUA .