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

Skip to content

Fix 'c' key intercepting search input in stacks view#270

Merged
clangenb merged 1 commit into
mainfrom
fix/259-stack-search-c-key
Mar 6, 2026
Merged

Fix 'c' key intercepting search input in stacks view#270
clangenb merged 1 commit into
mainfrom
fix/259-stack-search-c-key

Conversation

@eldara-cruncher
Copy link
Copy Markdown
Collaborator

@eldara-cruncher eldara-cruncher commented Mar 6, 2026

Summary

  • Fixes stack view - search error and add search to all views #259: pressing c while in / search mode on the stacks view opened the create stack dialog instead of appending to the search query
  • Root cause: the c shortcut handler was checked before the search mode guard in views/stacks/update.go
  • Fix: move the search mode guard above the c handler (swap two code blocks, no logic changes)

Test plan

  • go build compiles cleanly
  • go test ./views/stacks/... passes
  • go test ./... passes (all packages)
  • golangci-lint run ./... — 0 issues
  • Manual: open stacks view → press / → type a query containing c → verify it filters instead of opening create dialog

🤖 Generated with Claude Code

Move the search mode guard above the 'c' shortcut handler so that
typing 'c' while in '/' search mode appends to the filter query
instead of opening the create stack dialog.

Fixes #259

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@eldara-cruncher eldara-cruncher force-pushed the fix/259-stack-search-c-key branch from 51aed2c to 262076d Compare March 6, 2026 15:21
@clangenb clangenb merged commit 0661d91 into main Mar 6, 2026
11 checks passed
@eldara-cruncher eldara-cruncher deleted the fix/259-stack-search-c-key branch June 5, 2026 12:18
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.

stack view - search error and add search to all views

2 participants