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

feat: add multicursors-nvim plugin #610

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

Conversation

jahanson
Copy link

@jahanson jahanson commented Feb 8, 2025

multicursors-nvim with hydra dependency and config.

Added plugin and config for https://github.com/smoka7/multicursors.nvim

Sanity Checking

  • I have updated the changelog as per my changes
  • I have tested, and self-reviewed my code
  • Style and consistency
    • I ran Alejandra to format my code (nix fmt)
    • My code conforms to the editorconfig configuration of the project
    • My changes are consistent with the rest of the codebase
  • If new changes are particularly complex:
    • [ ] My code includes comments in particularly complex areas
    • [ ] I have added a section in the manual
    • [ ] (For breaking changes) I have included a migration guide
  • Package(s) built:
    • .#nix (default package)
    • .#maximal
    • .#docs-html (manual, must build)
  • Tested on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin

Add a 👍 reaction to pull requests you find important.

github-actions bot pushed a commit that referenced this pull request Feb 8, 2025
Copy link

github-actions bot commented Feb 8, 2025

🚀 Live preview deployed from 316d4a0

View it here:

Debug Information

Triggered by: jahanson

HEAD at: add-multicursors-nvim

Reruns: 192

@jahanson jahanson marked this pull request as ready for review February 8, 2025 18:27
@jahanson jahanson requested a review from NotAShelf as a code owner February 8, 2025 18:27
Comment on lines +5 to +7
options.vim.hydra = {
enable = mkEnableOption "Creating custom submodes and menus [nvimtools/hydra.nvim]";
};
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
options.vim.hydra = {
enable = mkEnableOption "Creating custom submodes and menus [nvimtools/hydra.nvim]";
};
options.vim.utility.hydra = {
enable = mkEnableOption "utility for creating custom submodes and menus [nvimtools/hydra.nvim]";
};

It's better to have plugins under specific categories instead polluting top-level vim.*. You'll also want to move this under plugins/utility.

Comment on lines +16 to +32
setupOpts = {
debug = false;
exit = false;
foreign_keys = null;
color = "red";
timeout = false;
invoke_on_body = false;
hint = {
show_name = true;
position = "bottom";
offset = 0;
float_opts = {};
};
on_enter = null;
on_exit = null;
on_key = null;
};
Copy link
Owner

Choose a reason for hiding this comment

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

Is there are a reason we're shipping those defaults? Does the plugin not set any defaults, or are the defaults unergonomic?


[jahanson](https://github.com/jahanson):

- Add [multicursors.nvim](https://github.com/smoka7/multicursors.nvim) to
Copy link
Owner

Choose a reason for hiding this comment

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

Worth mentioning the addition of the hydra.nvim dependency here`

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