mroot/yazi
1
0
Fork 0
mirror of https://github.com/sxyazi/yazi.git synced 2026-07-23 07:41:03 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

fix(help): search keybindings by key name in command palette

Browse source 
base cases covered for exact searching `on` command keys,
case-insensitive prefixes, and substring matches on key name or
description.
This commit is contained in:
Alastair Scheuermann 2026-06-23 14:15:45 -06:00 • committed by sxyazi
parent b1ee2e72ff
commit d41a42302c
No known key found for this signature in database
1 changed files with 42 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

48
yazi-core/src/help/help.rs
View file

@ -43,12 +43,48 @@ impl Help {
} else if self.keyword != kw {
let lowercased = kw.to_lowercase();
self.keyword = kw.to_owned();
self.bindings = KEYMAP
.chords(self.layer)
.iter()
.filter(|&c| c.desc_or_run().to_lowercase().contains(&lowercased))
.cloned()
.collect();
let exact_prefix = |on: &str| on.starts_with(kw);
let icase_prefix = |on: &str| on.to_lowercase().starts_with(&lowercased);
let all = KEYMAP.chords(self.layer);
// Priority 1: exact case-sensitive prefix (`y` -> `y`; `<C-` -> `<C-u>`)
let mut bindings: Vec<ChordArc> =
all.iter().filter(|c| exact_prefix(&c.on())).cloned().collect();
// Priority 2: case-insensitive prefix not in group 1 (`y` -> `Y`; `<e` -> `<Escape`)
bindings.extend(
all
.iter()
.filter(|c| {
let on = c.on();
!exact_prefix(&on) && icase_prefix(&on)
})
.cloned(),
);
// Priority 3: on() substring match (`enter` -> `<Enter>`, `<S-Enter>`)
let on_contains = |on: &str| on.to_lowercase().contains(&lowercased);
bindings.extend(
all
.iter()
.filter(|c| {
let on = c.on();
!icase_prefix(&on) && on_contains(&on)
})
.cloned(),
);
// Priority 4: desc-only match (`enter` -> `l` "Enter the child directory")
bindings.extend(
all
.iter()
.filter(|c| {
let on = c.on();
!icase_prefix(&on)
&& !on_contains(&on)
&& c.desc_or_run().to_lowercase().contains(&lowercased)
})
.cloned(),
);
self.bindings = bindings;
}
render!(self.scroll(0));