diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-10 13:15:48 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-10 13:15:48 -0500 |
| commit | f2c45a7146568bea7eea12270688c8c8ed814c86 (patch) | |
| tree | 866d32ca0484dea1d9414184988e2ae617ed0e16 /modules/custom-comments.el | |
| parent | 5eccbf7bd3c9780eee6170839dbf3224a32e9ef3 (diff) | |
| download | dotemacs-f2c45a7146568bea7eea12270688c8c8ed814c86.tar.gz dotemacs-f2c45a7146568bea7eea12270688c8c8ed814c86.zip | |
fix(dirvish): rename pw -> wp to clear quick-access key collision
Pressing `g' in dirvish (`dirvish-quick-access') failed with `Wrong type argument: command, (keymap (107 . transient:dirvish-quick-access::N))'. Transient auto-groups multi-character keys into nested prefix keymaps; when one entry's KEY is also the prefix of a longer entry's KEY, the shorter slot resolves to the auto-generated sub-prefix keymap rather than to the intended leaf command. The "wallpaper" entry on `pw' and the "project work" entry on `pwk' (added in 4ece1eb) collided. Rename `pw' to `wp'; `pwk' is now the only entry under the `pw'-prefix, so transient builds a clean leaf.
Add `tests/test-dirvish-config-quick-access.el' with five tests: four cover a small `test-dirvish-config--key-collisions' helper against synthetic inputs (collision detected, clean list, empty, single-entry), and one regression test asserts the live `dirvish-quick-access-entries' contains no shorter-key-is-prefix-of-longer-key pair.
Diffstat (limited to 'modules/custom-comments.el')
0 files changed, 0 insertions, 0 deletions