Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 55b4a33

Browse filesBrowse files
committed
fix(tui): fix burst capture key ordering and add tmux-safe alias
Ctrl+P was checked before Ctrl+Shift+P, so the narrower (single-frame) handler swallowed the event and the burst handler was unreachable. Swap the checks so the shift-modifier branch runs first. Additionally, tmux strips the shift bit from Ctrl+Shift+P in its default mode, so users inside tmux cannot trigger burst at all. Accept bare Shift+P as a tmux-safe alias for burst record. Ctrl+P continues to capture single frames.
1 parent aa86886 commit 55b4a33
Copy full SHA for 55b4a33

1 file changed

+4-4Lines changed: 4 additions & 4 deletions

File tree

Expand file treeCollapse file tree
Open diff view settings
Filter options
Expand file treeCollapse file tree
Open diff view settings
Collapse file

‎src/tui/App.tsx‎

Copy file name to clipboardExpand all lines: src/tui/App.tsx
+4-4Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -245,12 +245,12 @@ function AppContent() {
245245
]);
246246

247247
useKeyboard((key) => {
248-
if (key.ctrl && key.name === "p") {
249-
handleCaptureFrame();
248+
if ((key.ctrl && key.shift && key.name === "p") || (key.shift && key.name === "p")) {
249+
handleBurstRecord();
250250
return;
251251
}
252-
if (key.ctrl && key.shift && key.name === "p") {
253-
handleBurstRecord();
252+
if (key.ctrl && key.name === "p") {
253+
handleCaptureFrame();
254254
return;
255255
}
256256

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.