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

Allow disabling automatic keymaps #341

Closed
wants to merge 2 commits into from

Conversation

will-lynas
Copy link
Contributor

@will-lynas will-lynas commented Aug 28, 2024

This adds the ability to set certain keymaps to false (instead of a string) in the config to disable their creation, and allows users to create their own keymaps.

This is useful in case they want to set their own desc or other opts.

It also exposes the functions to toggle hints and debug so they can be used in keymaps.

I'm not sure where the best place to document this would be. Feedback is appreciated!

@will-lynas
Copy link
Contributor Author

will-lynas commented Aug 28, 2024

Stylua seems to be failing from an unrelated commit. See #342

If a keymap is set to false or nil in the config, don't attempt to set
it.
Copy link
Collaborator

@aarnphm aarnphm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm let me propose something.

@aarnphm
Copy link
Collaborator

aarnphm commented Aug 29, 2024

See #346

@aarnphm aarnphm closed this Aug 29, 2024
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.

2 participants