Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Be more lenient when pressing TAB and nothing matches #193

Merged
merged 3 commits into from
Sep 5, 2020
Merged

Be more lenient when pressing TAB and nothing matches #193

merged 3 commits into from
Sep 5, 2020

Conversation

d125q
Copy link
Contributor

@d125q d125q commented Sep 4, 2020

Currently, an error occurs when TAB is pressed and no candidate
matches the minibuffer contents. Moreover, the minibuffer contents
are erased.

This commit makes the behavior more lenient and similar to the default
minibuffer completion -- the bell is rung a “No match” message is
displayed. Also, ‘completion-fail-discreetly’ is respected so none of
the aforementioned things are done if it is non-nil.

Currently, an error occurs when TAB is pressed and no candidate
matches the minibuffer contents.  Moreover, the minibuffer contents
are erased.

This commit makes the behavior more lenient and similar to the default
minibuffer completion -- the bell is rung a “No match” message is
displayed.  Also, ‘completion-fail-discreetly’ is respected so none of
the aforementioned things are done if it is non-nil.
@clemera clemera merged commit c0f0a7b into radian-software:master Sep 5, 2020
@clemera
Copy link
Collaborator

clemera commented Sep 5, 2020

Very nice, thanks!

raxod502 added a commit that referenced this pull request Sep 19, 2020
@@ -131,6 +131,10 @@ The format is based on [Keep a Changelog].
an error would be thrown ([#152]).
* When completing filenames and a match is required, non-normalized
paths (e.g., `~/Documents//etc/hosts`) are accepted ([#190]).
* Pressing TAB when nothing matches shows a “No match” message in the
minibuffer instead of signaling an error and erasing the minibuffer
contents ([#193]). If ‘completion-fail-discreetly’ is non-nil,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This style of quotes won't render correctly in Markdown, so I updated master to use regular backticks on both sides.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, I did not notice those quotes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants