Skip to content

Update orderless-escapable-split-on-space to support other characters#197

Open
dadinn wants to merge 1 commit intooantolin:masterfrom
dadinn:develop
Open

Update orderless-escapable-split-on-space to support other characters#197
dadinn wants to merge 1 commit intooantolin:masterfrom
dadinn:develop

Conversation

@dadinn
Copy link
Copy Markdown

@dadinn dadinn commented Apr 27, 2026

Allows customising the escapable component separator character to be other than just a space. This feature was originally requested/discussed here.

Introduces a new customisable variable orderless-escapable-separator, which allows any character to be used as separator, not just the default space character (to be backwards compatible).

Example with corfu:
Screenshot From 2026-04-27 07-12-25

Example with vertico:
Screenshot From 2026-04-27 07-15-23

The above examples use the following orderless-matching-styles:

(setq orderless-matching-styles
   '(orderless-flex orderless-regexp))

Allow customizing the escapable component separator character
to be other than just a space.
@minad
Copy link
Copy Markdown
Collaborator

minad commented Apr 27, 2026

Thanks for the proposal. I am a bit skeptical about this addition, since I find it a bit too special. I am unsure if many users will need it, since it hasn't come up in discussion yet, except the linked issue. One can always write a custom splitting function if really needed. Can you show a real use case where we need this? In Corfu and in minibuffer completion in general the space separator works well.
@oantolin What do you think?

@minad
Copy link
Copy Markdown
Collaborator

minad commented Apr 27, 2026

It seems @oantolin had already rejected this feature request in #102 (comment). I think this still holds without further evidence.

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.

2 participants