Skip to content

Start video at text-cursor #106

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
murezzda opened this issue Mar 1, 2019 · 1 comment
Closed

Start video at text-cursor #106

murezzda opened this issue Mar 1, 2019 · 1 comment
Assignees
Labels
Enhancement a request for improvement

Comments

@murezzda
Copy link
Contributor

murezzda commented Mar 1, 2019

Is your feature request related to a problem? Please describe.
In conjunction with #97, it would be great if the player can be started from the current word at the cursor position. This would allow for rapid navigation inside the video via keyboard.

Describe the solution you'd like
If the user starts the video via the alt+k shortcut, the video starts from the current cursor position instead of the current video position.

Additional context
As this feature alters the behaviour of the alt+k shortcut significantly, it might be good to put an option in the option menu to enable/disable this behaviour. Alternatively, another shortcut could be introduced for this feature, but I think this might increase the complexity of the application too much.

@murezzda murezzda added the Enhancement a request for improvement label Mar 1, 2019
@pietrop
Copy link
Contributor

pietrop commented Mar 22, 2019

Thanks @murezzda since we closed #97 and parked it for now, closing this one as well. We are looking to consolidate the component before adding new feature, but happy to revisit at a later stage

@pietrop pietrop closed this as completed Mar 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement a request for improvement
Projects
None yet
Development

No branches or pull requests

2 participants