Skip to content

feat: add enableSeekPause option#82

Open
EastSun5566 wants to merge 3 commits intoctd1500:masterfrom
EastSun5566:fix/keep-playing-state-after-seek
Open

feat: add enableSeekPause option#82
EastSun5566 wants to merge 3 commits intoctd1500:masterfrom
EastSun5566:fix/keep-playing-state-after-seek

Conversation

@EastSun5566
Copy link
Copy Markdown

Hi @ctd1500 ,

Thanks for the great plugin!

After I press the right/left arrow keys to seek video will cause a pause when it is playing.
I think just using the currentTime method to seek will keep playing state. No need to keep track current playing state.

Please let me know what you think. Thank you!

@ctd1500
Copy link
Copy Markdown
Owner

ctd1500 commented Dec 1, 2022

The pause/play is there for consistency with how the seekbar works when using the mouse: #41.

You could add a new option like enableSeekPause, default to true for compatibility.

@EastSun5566
Copy link
Copy Markdown
Author

Thank you for explaining that!

enableSeekPause sounds like a good option. I will turn this PR into a draft and then implement it.

@EastSun5566 EastSun5566 changed the title fix: keep playing state after seek WIP: feat: add enableSeekPause option Dec 1, 2022
@EastSun5566 EastSun5566 force-pushed the fix/keep-playing-state-after-seek branch from 68458e6 to 63ef759 Compare December 1, 2022 16:25
@EastSun5566 EastSun5566 changed the title WIP: feat: add enableSeekPause option feat: add enableSeekPause option Dec 24, 2022
@EastSun5566
Copy link
Copy Markdown
Author

Hi @ctd1500 ,

Sorry for being a little bit late.

I added the enableSeekPause option for the previous discussion & also update README.md.
Please take a look & let me know what you think!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants