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

Enable ido-everywhere with function call #19

Merged
merged 1 commit into from
Feb 10, 2021
Merged

Enable ido-everywhere with function call #19

merged 1 commit into from
Feb 10, 2021

Conversation

susam
Copy link
Contributor

@susam susam commented Feb 10, 2021

Prior to this change, (setq ido-everywhere t) has no effect. For
example, if we type M-x ediff-files RET, Ido mode does not work in
the minibuffer. The documentation for ido-everywhere accessible via
C-h v ido-everywhere RET has the following note regarding this
behaviour:

Setting this variable directly does not take effect;
either customize it (see the info node ‘Easy Customization’)
or call the function ‘ido-everywhere’.

This change fixes this issue by calling the ido-everywhere function.
With this fix, Ido mode now works with M-x ediff-files RET.

Prior to this change, `(setq ido-everywhere t)` has no effect. For
example, if we type `M-x ediff-files RET`, Ido mode does not work in
the minibuffer. The documentation for `ido-everywhere` accessible via
`C-h v ido-everywhere RET` has the following note regarding this
behaviour:

    Setting this variable directly does not take effect;
    either customize it (see the info node ‘Easy Customization’)
    or call the function ‘ido-everywhere’.

This change fixes this issue by calling the `ido-everywhere` function.
With this fix, Ido mode now works with `M-x ediff-files RET`.
@Shinmera
Copy link
Member

Might have been a change in ido that I didn't catch up with, then. Thanks!

@Shinmera Shinmera merged commit 9017d29 into portacle:master Feb 10, 2021
@susam susam deleted the ido-everywhere branch February 10, 2021 11:11
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