-
-
Notifications
You must be signed in to change notification settings - Fork 340
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
ivy complete integration. #2321
Comments
Thanks for the info. I also went through the thread on emacs-devel. It's really unfortunate that just two people managed to take over the whole narrative based on their personal preferences. I can respect Helm, it's quite advanced and well maintained. But comparing those hacks in Ido and telling that they're clearly better than Ivy is very misinformed. Also stating that Icomplete is somehow better or simpler than Ivy: Ivy grew from Icomplete by adding the features users wanted. Just check out an early commit of Ivy and you'll see Icomplete. I'll go through the linked thread later and see what can be done. |
I didn't say it's better? If I did, I apologize. How could I? I don't use Ivy at all: I can't judge. It does seem kinda irrefutable that it's much much simpler tho (like at least 10x less lines). Simply because of its decision to have less features, or to have them provided by other components. |
I believe the other person said that Ido is clearly better. It seems they did not look into
Arguable, but not irrefutable, IMO. See commit 6a874a4. There, ivy.el is 200 lines of code, basically a cleaned up icomplete.el. The "other components" trick can be arguably counted towards icomplete line count. |
lol, you see there is thing called time and these things called past and present. now seriously, you were kidding, right? I meant "now" of course, present tense. the fact that it was simpler then has no relation on the fact that it is more complicated now.
This "trick" is called software engineering. I love such trickery, quite the fan. I'm kidding, of course, don't take this wrong, please. Superlatively amazing work on Ivy, judging by your fandom! I hope you can make it work with Emacs's completion interfaces, just like Helm did, so we can all enjoy it bit by bit and rave about that one initial commit time where our files were empty and infinite times simpler than they are today. |
Just a note. I did say that myself, and you are right to complain. I have no idea, as I said earlier, I never tried your program. Ido's interface seems better (because it seems simpler). |
I should mention that In any case I hope @abo-abo can integrate whats is missing to make it better. |
@Ergus @joaotavora I read through the whole thread, but I can't quite make out what's missing. Could someone enlighten me with a small reproduction scenario? |
Sorry I use ivy very much but I don't know anything about the new functioalities in the completion system maybe @joaotavora could make an abstract of what was added to Helm and then we can see the missing functionalities. So far there shouldn't be too much missing; probably just details. As ivy is based on icomplete, only the new changes will be missing. |
I think Helm's author is the person most qualified to do that. I just know, from second hand knowledge, that it now works acceptably in SLY. Also, @abo-abo I will add you to an off-list discussion about changes to completion system. |
Probably makes sense to mention @thierryvolpiatto to see if he can add something useful for @abo-abo, then. :) |
I will try to write something in helm wiki about completion-styles implementation, not yet though because the API may change soon and I have few time. |
@thierryvolpiatto Thanks for the offer. I will ask for help when I have more time to look into this. Looks like a lengthy project. |
Hi:
Recently the emacs developers were helping the helm developers to add more integration with all the internal completion infrastructure. Probably part of it is already into ivy (as it is bases in icomplete), but it worth to give it a look if there is something missing in ivy/counsel that can be added/improved.
The thread is: (emacs-helm/helm#2165)
I made some references to this in the mailing list today.
The text was updated successfully, but these errors were encountered: