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

replace custom ivy hydra-menu by original hydra-menu #13377

Closed
wants to merge 4 commits into from
Closed

replace custom ivy hydra-menu by original hydra-menu #13377

wants to merge 4 commits into from

Conversation

dalanicolai
Copy link
Contributor

@dalanicolai dalanicolai commented Mar 14, 2020

Replaced the custom ivy hydra menu with the original (and updated) ivy hydra menu. Additionally checked parts of the documentation that required modification and applied them accordingly (no modifications were needed, only some additions)

@nixmaniack
Copy link
Contributor

👍 For this change, my only gripe is the missing s-M-SPC–an alternate for M-SPC–according the convention. We should, ideally, add them.

@smile13241324
Copy link
Collaborator

Thank you for contributing to spacemacs 💜, I have squashed and cherry picked your commits into develop.

@dalanicolai next time please take care that your PR has only one commit. To do so either force push on your PR branch or make single commits and squash them in the end.

@dalanicolai dalanicolai deleted the replace_custom_ivy_hydra_menu_with_updated_original branch March 18, 2020 19:13
@dalanicolai
Copy link
Contributor Author

Yeah I am learning git... and sometimes some things happen that I do not understand yet... anyway I will check things again and next time think about the squash

@j-martin
Copy link
Contributor

I suspect this PR broke the completion if not using the ivy layer. #13395

@dalanicolai dalanicolai mentioned this pull request Mar 21, 2020
smile13241324 pushed a commit that referenced this pull request Mar 21, 2020
Ivy-hydra is autoloaded so the require statement here is unnecessary. Because this require (commented out) statement was here before I uncommented it in the last PR (#13377), I leave it here (commented out) as it was before that PR.
epsilon-0 pushed a commit to epsilon-0/spacemacs that referenced this pull request Apr 21, 2020
Ivy-hydra is autoloaded so the require statement here is unnecessary. Because this require (commented out) statement was here before I uncommented it in the last PR (syl20bnr#13377), I leave it here (commented out) as it was before that PR.
aam-at pushed a commit to aam-at/spacemacs that referenced this pull request Mar 23, 2021
Ivy-hydra is autoloaded so the require statement here is unnecessary. Because this require (commented out) statement was here before I uncommented it in the last PR (syl20bnr#13377), I leave it here (commented out) as it was before that PR.
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.

4 participants