Skip to content

Added keyboard shortcuts to the video editor#178

Open
KoopaCode wants to merge 5 commits intosiddharthvaddem:mainfrom
KoopaCode:main
Open

Added keyboard shortcuts to the video editor#178
KoopaCode wants to merge 5 commits intosiddharthvaddem:mainfrom
KoopaCode:main

Conversation

@KoopaCode
Copy link
Contributor

Pull Request Template

Description

Added keyboard shortcuts to the video editor: spacebar toggles play/pause, left/right arrow keys seek ±1 second.

Motivation

Allows faster and more video navigation using the keyboard.

Type of Change

  • New Feature
  • Bug Fix
  • Refactor / Code Cleanup
  • Documentation Update
  • Other (please specify)

Related Issue(s)

N/A

Screenshots / Video

Video (if applicable):

https://i.nulz.host/vgLCH8Rkrq.gif

Testing

  1. Open the video editor.
  2. Press spacebar to toggle play/pause.
  3. Use left/right arrow keys to seek backward/forward by 1 second.

Checklist

  • I have performed a self-review of my code.
  • I have added any necessary screenshots or videos.
  • I have linked related issue(s) and updated the changelog if applicable.

Thank you for contributing!

@FabLrc FabLrc mentioned this pull request Mar 1, 2026
8 tasks
@FabLrc
Copy link
Contributor

FabLrc commented Mar 1, 2026

Hey, thanks for your work ! A configurable keyboard shortcuts system #172 will be soon merged into main, if you want, you can already adapt your PR to this new feature.

@siddharthvaddem
Copy link
Owner

#172 has been merged. If you can adapt your PR to sync with these new changes, that would be great 🙏

@siddharthvaddem
Copy link
Owner

Actually I think these would be fixed anyways, spacebar toggle is the de facto pause/play and left right keys make sense for moving as well. I think this should be good to merge once you resolve the conflicts.

@8sami
Copy link

8sami commented Mar 5, 2026

please also add del key for deleting the selected element

@8sami
Copy link

8sami commented Mar 5, 2026

also, the undo and redo dont work as they add a zoom instead pressing the Z key

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.

4 participants