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

company-flx seems to be required. #3

Open
joostkremers opened this issue Mar 2, 2021 · 4 comments
Open

company-flx seems to be required. #3

joostkremers opened this issue Mar 2, 2021 · 4 comments

Comments

@joostkremers
Copy link

This is really a great idea, I'm usually futzing with the X-Sampa input method...

Unfortunately, it seems to have a hard dependency on company-flx, because without it, it doesn't really work. I get list of completion candidates that I can scroll through, but typing part of the name does not narrow the list of candidates.

I don't know enough about company to know if it can be made to work without company-flx, but if not, you should probably mention this in the README.

@mguzmann
Copy link
Owner

mguzmann commented Mar 2, 2021

The alternative to using company-flx is having to type the exact name of the candidate, which I find super unpractical. Not sure what the solution would be. Do you have any ideas? BTW, this is now in melpa and the offical repo is over in gitlab. I'll keep this as a mirrow and update it soon.

@joostkremers
Copy link
Author

The alternative to using company-flx is having to type the exact name of the candidate, which I find super unpractical. Not sure what the solution would be. Do you have any ideas?

I agree it's much more practical with flx, but an alternative without it would be if similar characters start with the same substring. Then you can use TAB to narrow the list of candidates and M-n / M-p to get to the desired symbol once the list is sufficiently small. Currently, that's not possible because the candidate strings have spaces in them.

I'm not sure it's worth the trouble, though. If you'd accept a PR, I might look into it some time next week.

BTW, this is now in melpa

Yeah, I learnt about the package when I updated my package list. :-)

@mguzmann
Copy link
Owner

mguzmann commented Mar 2, 2021

I'm not sure it's worth the trouble, though. If you'd accept a PR, I might look into it some time next week.

Sure, but please use the gitlab repo: https://gitlab.com/mguzmann89/company-ipa/

@joostkremers
Copy link
Author

Sure, but please use the gitlab repo: https://gitlab.com/mguzmann89/company-ipa/

Will do.

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