Skip to content

Comments

Add keyboard shortcuts for Scene Cover generation#5984

Merged
WithoutPants merged 4 commits intostashapp:developfrom
n0ld069:kb_shortcuts_generate
Nov 10, 2025
Merged

Add keyboard shortcuts for Scene Cover generation#5984
WithoutPants merged 4 commits intostashapp:developfrom
n0ld069:kb_shortcuts_generate

Conversation

@n0ld069
Copy link
Contributor

@n0ld069 n0ld069 commented Jun 30, 2025

DESCRIPTION:

I like using custom cover screenshots for my scenes. When playing a video, I will often spot a good place to pause to generate the screenshot, but then the button to "Generate screenshot from current" is buried under a submenu in the scene details tab and requires a few clicks.

This adds 1 shortcut to generate the screenshot at current time, and 1 screenshot to re-generate a default screenshot. I tried to ensure the shortcuts follow the conventions of existing shortcuts and do not conflict with any existing shortcuts.

Ideally a system to define any custom shortcuts could be cool, but that's a much larger effort.

CHANGES:

  • Add 'c c' shortcut to generate screenshot at current time
  • Add 'c d' shortcut to generate default screenshot
  • Update keyboard shortcuts documentation

- Add 'c c' shortcut to generate screenshot at current time
- Add 'c d' shortcut to generate default screenshot
- Update keyboard shortcuts documentation
Copy link
Collaborator

@DogmaDragon DogmaDragon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documentation included. 👍

@n0ld069
Copy link
Contributor Author

n0ld069 commented Jul 1, 2025

(yes, I'm using Claude Code. If this typescript fix is not the most appropriate fix, please advise.)

Add explicit braces to arrow functions to satisfy TypeScript's strict mode
requirements for async function calls.
@n0ld069 n0ld069 force-pushed the kb_shortcuts_generate branch from 230f6bc to d4e141f Compare July 1, 2025 19:28
@n0ld069
Copy link
Contributor Author

n0ld069 commented Jul 11, 2025

Sorry for all the pushes, I should have closed the PR while I worked this out.

@WithoutPants WithoutPants added the improvement Something needed tweaking. label Nov 10, 2025
@WithoutPants WithoutPants merged commit 12a9a0b into stashapp:develop Nov 10, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Something needed tweaking.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants