Add option to repeat the current video#37
Conversation
|
Thanks for the contribution, @hymansun — but I don't think I can merge this one. Let me explain the reasoning so you can decide what to do with the branch. Macify is built around random playback as a core behavior, not as a default that users might want to override:
That's just one example of the conflict — and I want to be upfront that even if that specific case were resolved, I still wouldn't want to add a "repeat current video" option, because it cuts against the product's core idea. If this is something you personally want, I'd suggest keeping the change on your own fork. Really appreciate the effort you put in here, but I'll have to pass on merging. |
|
This is an option; if the user doesn't check it, the next video will play by default. I think it's better to give the user the choice, but of course, I respect your opinion. |
Summary
repeatCurrentVideosetting, defaulting to off<video>when enabled while preserving the existing auto-next behavior when disabledTest
npm run build