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

Ido completion incomaptible with new completion for describe-function and describe-variable #111

Closed
DarwinAwardWinner opened this issue Jun 23, 2016 · 7 comments

Comments

@DarwinAwardWinner
Copy link
Owner

DarwinAwardWinner commented Jun 23, 2016

From #110 (comment):

Note that in [adding overrides for describe-function and describe-variable], you're breaking the new feature: automatic loading of packages based on known variable/function name prefixes.

I need to figure out whether ido completion can be made to work with this new feature, and if not, just leave ido disabled for these functions.

@DarwinAwardWinner
Copy link
Owner Author

Ido completion needs to know all the possible completions in advance, so the only way to make this work would be to pre-load all known elisp files, which is not an acceptable solution.

gdobbins added a commit to gdobbins/ido-ubiquitous that referenced this issue Feb 14, 2017
This patch enables describe-variable and describe-function to use ido
while also having the feature of automatically loading relevant libs
based on the prefix of the completion. This fixes issue DarwinAwardWinner#111
@DarwinAwardWinner
Copy link
Owner Author

Reopening this since #118 offers a potential solution.

@theophilusx
Copy link

Just thought I'd add to this as I've now encountered this issue when using mu4e. Once you run mu4e, C-h v will no longer work. See djcb/mu#1078

@DarwinAwardWinner
Copy link
Owner Author

DarwinAwardWinner commented May 15, 2017

I don't think this is the same issue. This issue is about describe-variable and describe-function not having ido completion. If you're encountering errors, please open a separate issue, with the exact sequence of commands to reproduce the error starting from emacs -Q.

@DarwinAwardWinner
Copy link
Owner Author

Hi @theophilusx, I've opened a separate issue, #119, to track your error. Could you please add directions on how to reproduce the error there?

@theophilusx
Copy link

theophilusx commented May 16, 2017 via email

DarwinAwardWinner pushed a commit that referenced this issue May 17, 2017
This patch enables describe-variable and describe-function to use ido
while also having the feature of automatically loading relevant libs
based on the prefix of the completion. This fixes issue #111
@DarwinAwardWinner
Copy link
Owner Author

DarwinAwardWinner commented Jun 1, 2017

Fixed in the series of commits 320a00a...65a291a.

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

No branches or pull requests

2 participants