Skip to content

fix(default): prioritize neotree/treemacs sidebar bindings over dirvish#8787

Open
LemonBreezes wants to merge 1 commit into
doomemacs:masterfrom
LemonBreezes:fix/dirvish-sidebar-binding-precedence
Open

fix(default): prioritize neotree/treemacs sidebar bindings over dirvish#8787
LemonBreezes wants to merge 1 commit into
doomemacs:masterfrom
LemonBreezes:fix/dirvish-sidebar-binding-precedence

Conversation

@LemonBreezes

@LemonBreezes LemonBreezes commented May 30, 2026

Copy link
Copy Markdown
Contributor

I think most people prefer to use Treemacs or Neotree over Dirvish Side if they explicitly have it enabled in their config since these keybindings are the primary entry-points for those packages.

Move the (:when (modulep! :emacs dired +dirvish)) block ahead of the
neotree and treemacs blocks for the SPC o sidebar bindings. Since later
map! definitions override earlier ones for the same key, placing dirvish
first lets the neotree/treemacs 'p'/'P' bindings take precedence when
those modules are enabled alongside dirvish.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@LemonBreezes LemonBreezes changed the title fix(default): prioritize dirvish sidebar bindings over neotree/treemacs fix(default): prioritize neotree/treemacs sidebar bindings over dirvish May 30, 2026
@LemonBreezes LemonBreezes force-pushed the fix/dirvish-sidebar-binding-precedence branch from 39bbdd7 to 0ea84e1 Compare May 30, 2026 19:57
@LemonBreezes LemonBreezes marked this pull request as ready for review May 30, 2026 19:59
@LemonBreezes LemonBreezes requested a review from a team as a code owner May 30, 2026 19:59
@hlissner hlissner added is:bug Something isn't working as intended re:keybinds Changes to or discussion about Doom's keybinds module:config/default Pertains to Doom's :config default module was:moved Is, was, or will be addressed elsewhere labels May 30, 2026
@hlissner hlissner added this to the modules v26.07 milestone May 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

is:bug Something isn't working as intended module:config/default Pertains to Doom's :config default module re:keybinds Changes to or discussion about Doom's keybinds was:moved Is, was, or will be addressed elsewhere

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants