Skip to content

docs(guides): refresh video editor cheatsheet with I/O work-area shortcuts#824

Open
calcarazgre646 wants to merge 1 commit into
heygen-com:mainfrom
calcarazgre646:docs/cheatsheet-work-area-shortcuts
Open

docs(guides): refresh video editor cheatsheet with I/O work-area shortcuts#824
calcarazgre646 wants to merge 1 commit into
heygen-com:mainfrom
calcarazgre646:docs/cheatsheet-work-area-shortcuts

Conversation

@calcarazgre646
Copy link
Copy Markdown
Contributor

Summary

Refreshes docs/guides/video-editor-cheatsheet.mdx so the Preview Shortcuts section matches what's actually wired up in the Studio today. The single flat table is split into the same four groups the in-app panel uses (Playback, Work area, View, Application), with each row traced to the source handler so the doc stays in sync with the code.

New rows (previously undocumented)

Corrections to existing rows

  • Arrow keys step 1 frame and Shift+Arrow steps 10 frames — the previous "1 second" / "5 seconds" wording was wrong (see stepFrames(e.shiftKey ? -10 : -1) in usePlaybackKeyboard.ts).
  • "Seek bar focused" qualifier dropped — the keydown listener is window + iframe-level, gated only by shouldIgnorePlaybackShortcutEvent for editable targets.

Narrative additions

Closes #812.

Test plan

  • npx mint validate — success build validation passed.
  • npx mint broken-links — no broken links found.
  • npx tsx scripts/sync-schemas.ts --check — registry schemas in sync.
  • Cross-checked every row against the source: usePlaybackKeyboard.ts, useAppHotkeys.ts, PlayerControls.tsx (SHORTCUT_SECTIONS), NLEPreview.tsx.
  • Conventional commit + lefthook commitlint pass.

Happy to adjust grouping, naming (e.g. "Work area" vs "Range"), or table density if the maintainers prefer a different style.

…tcuts

The Preview Shortcuts section was missing the shortcuts shipped in heygen-com#710
and heygen-com#811: J/K/L shuttle, I/O work-area markers with A/E jumps, the
Cmd/Ctrl+Scroll preview zoom-at-cursor, and undo/redo. The arrow-key
entries were also wrong — they step by frames, not seconds.

Reorganise the table into the same groups the in-app `⌨` panel uses
(Playback, Work area, View, Application) and source each row from
`usePlaybackKeyboard.ts`, `useAppHotkeys.ts`, and the `NLEPreview` wheel
handler so the doc tracks what the Studio actually does. Mention that
the loop respects the work area when in/out are set, per heygen-com#811.

Closes heygen-com#812
@calcarazgre646
Copy link
Copy Markdown
Contributor Author

Hi @miguel-heygen — apologies, I missed that #812 was already assigned to @O1Anas with your offer to merge their PR (I only checked the issue via gh issue list, which doesn't surface comments). Happy to close this so @O1Anas can take it as originally planned. If for any reason that doesn't happen and this is useful as a fallback, glad to leave it open; otherwise just say the word and I'll close.

@O1Anas
Copy link
Copy Markdown

O1Anas commented May 14, 2026

thank you @calcarazgre646 ! the changes you made are better than I could've made 💯

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.

(docs) Update Video Editor Cheatsheet with latest shortcut additions

3 participants