Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: change app exit key from q to Ctrl+Q#328

Merged
clangenb merged 2 commits into
mainfrom
325-change-exit-key
Apr 15, 2026
Merged

feat: change app exit key from q to Ctrl+Q#328
clangenb merged 2 commits into
mainfrom
325-change-exit-key

Conversation

@eldara-cruncher
Copy link
Copy Markdown
Collaborator

@eldara-cruncher eldara-cruncher commented Apr 14, 2026

Summary

  • Disable bare q as quit/back key to prevent accidental exits from root view
  • Ctrl+Q is the primary quit key (shown in helpbar), Ctrl+C remains as standard terminal fallback
  • Esc is now the sole go-back key across all views
  • Add :q / :quit commands for k9s parity
  • Update all help text, helpbar entries, per-view key handlers, and tests

Closes #325

Test plan

  • q does nothing on any view (root and nested)
  • Ctrl+Q exits from root (stacks) view
  • Ctrl+Q exits from nested view (e.g. services → logs)
  • Ctrl+C exits from any view
  • Esc goes back from nested views, does nothing at root
  • :q and :quit commands exit the app
  • Typing q in search/filter input works normally
  • Helpbar shows <ctrl+q> quit globally
  • Per-view helpbar shows <esc> back/close
  • ? help pages show updated keys
  • go test ./... passes
  • golangci-lint run ./... passes

🤖 Generated with Claude Code

Disable bare `q` as quit/back key to prevent accidental exits.
Ctrl+Q is now the primary quit key, Ctrl+C remains as standard
terminal fallback. Esc is the sole go-back key. Add :q/:quit
commands for k9s parity.

Closes #325

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@clangenb clangenb requested a review from mosonyi April 14, 2026 08:00
Copy link
Copy Markdown
Collaborator

@mosonyi mosonyi left a comment

Choose a reason for hiding this comment

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

LGTM

Provides a single source of truth for canonical key names so that
extension repos (e.g. swarmcli-be) can reference the contract
rather than tracking raw string literals across repo boundaries.

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
@clangenb clangenb merged commit 6648ba4 into main Apr 15, 2026
10 of 11 checks passed
@eldara-cruncher eldara-cruncher deleted the 325-change-exit-key branch June 5, 2026 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change App Exit method

3 participants