Skip to content

Bind this-command in consult-dir-jump-file#48

Open
homonyms wants to merge 1 commit intokarthink:masterfrom
homonyms:fix/jump-file-this-command
Open

Bind this-command in consult-dir-jump-file#48
homonyms wants to merge 1 commit intokarthink:masterfrom
homonyms:fix/jump-file-this-command

Conversation

@homonyms
Copy link
Copy Markdown

consult-dir-jump-file executes the target command (like consult-find/fd) via a timer, but it does not bind this-command to that target.

This causes an issue with vertico-multiform: since this-command is empty, Vertico cannot match the specific style configured for the actual jump command. For example, a custom display for consult-find will not be triggered.

  • (consult-dir-jump-file): Bind this-command to
    consult-dir-jump-file-command within the deferred lambda. This ensures that
    the jump command (e.g., consult-find) identifies itself correctly, allowing
    completion UIs to adapt their styling accordingly.

* consult-dir.el (consult-dir-jump-file): Bind `this-command` to
`consult-dir-jump-file-command` within the deferred lambda. This
ensures that the jump command (e.g., `consult-find`) identifies itself
correctly, allowing completion UIs to adapt their styling accordingly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant