holm/nushell
1
1
Fork 0
forked from extern/nushell
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Exit explore on ctrl+c/d/q (#10257)

Browse Source 
Currently, `ctrl+z` is the command to exit `explore` no matter where you
are in the UI. IMO this is a bit unintuitive since that's usually used
to suspend a process.

After this change, `ctrl+c`, `ctrl+d`, and `ctrl+q` all work to exit
`explore`.

I think these are all shortcuts that users might try when attempting to
exit `explore`, and I think we might as well handle them all.
This commit is contained in:
Reilly Wood 2023-09-07 10:47:17 -07:00 committed by GitHub
parent b6189879e3
commit f021be623e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
crates/nu-explore/src/pager/mod.rs
View File

@ -781,14 +781,13 @@ fn handle_event<V: View>(
} }
fn handle_exit_key_event(key: &KeyEvent) -> bool { fn handle_exit_key_event(key: &KeyEvent) -> bool {
matches!( if key.modifiers == KeyModifiers::CONTROL {
key, // these are all common things people might try, might as well handle them all
KeyEvent { if let KeyCode::Char('c') | KeyCode::Char('d') | KeyCode::Char('q') = key.code {
code: KeyCode::Char('z'), return true;
modifiers: KeyModifiers::CONTROL,
..
} }
) }
false
} }
fn handle_general_key_events1<V>( fn handle_general_key_events1<V>(