Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

fix(tui): fix broken /export toggling#16443

Merged
nexxeln merged 1 commit intoanomalyco:devfrom
natewill:fix/export-toggle-clean
Mar 7, 2026
Merged

fix(tui): fix broken /export toggling#16443
nexxeln merged 1 commit intoanomalyco:devfrom
natewill:fix/export-toggle-clean

Conversation

@natewill
Copy link
Contributor

@natewill natewill commented Mar 7, 2026

Issue for this PR

Closes #16437

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

/export dialog toggling/selection does not work for terminals that use the kitty keyboard protocol (ex. vscode integrated terminal). This is because the dialog keybinding expects the "space" token for toggling while the vscode integreated terminal emits a " " token instead. We fixed this by adding if (evt.name === "space" || evt.name === " ") to dialog-export-options.

How did you verify your code works?

Tested on iTerm2, kitty, and the vscode integrated terminal and ran
bun test test/keybind.test.ts
bun run typecheck
bun turbo typecheck

Screenshots / recordings

Before fix

Screen.Recording.2026-03-07.at.12.27.33.AM.mov

After fix

Screen.Recording.2026-03-07.at.12.50.48.AM.mov

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@nexxeln nexxeln merged commit 5b5b791 into anomalyco:dev Mar 7, 2026
14 checks passed
@natewill natewill mentioned this pull request Mar 7, 2026
6 tasks
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.

[BUG]: /export toggle does not work with the vscode integrated terminal

2 participants