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

separate out completion ui from editing ui #10

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

oxalorg
Copy link
Member

@oxalorg oxalorg commented Jun 23, 2022

Currently all user interface code is present in corgi-editor.el, but the problem is there are two (maybe more) types of user interface changing code.

One is all the editing related code, so things like Avy, Undo, Evil, etc.

Other is menu completion/interactive UI, like Ivy, Counsel etc.

I've separated out the completition ui into a separate package so it's optional and thus easier to replace (with vertico/consult etc)

  • I've edited the Package-Requires by hand, is there a way to auto generate it?
  • Maybe projectile should not be a part of completion ui, but it didn't feel right in editor either

@SignSpice
Copy link

Curious if there is something this is waiting for to be merged. Using this patch locally.

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