fix: forward / and : to active inputs#302
Conversation
…ng them The `:` and `/` key handlers in app/update.go ran before the commandInput/searchInput forwarding logic, so those characters were intercepted and swallowed when an input was active. This made it impossible to type `/` or `:` in the search bar, command bar, or view-internal ctrl+f search. Fixes #300 Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
8f1fd7a to
609e3d9
Compare
mosonyi
left a comment
There was a problem hiding this comment.
In log view, I can search now for '/'.
I don't like the idea of searching for something, then all the other logs are hidden.
Better to show everything but highlight the search text.
You can still have the regular highlighted search with the s-key. |
At the log view the 's' is the Autoscroll off/on |
|
Then check for yourself, there is a key XD |
Summary
/and:as text to commandInput, searchInput, and view-internal search (ctrl+f) instead of swallowing them or triggering actionsapp/update.gowhere the:and/handlers ran before the input-forwarding logicFixes #300
Test plan
go test ./app/... -run "TestSlash|TestColon"— 5 new tests passgo test ./...— full suite passes/, type/and:— characters appear:, type:and/— characters appear:— character appears🤖 Generated with Claude Code