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(*): overhaul, new select_* functions #6

Closed
wants to merge 1 commit into from

Conversation

hinell
Copy link

@hinell hinell commented Oct 12, 2023

overhaul-demo.mov
  • feat(commands): clearly split normal mode and select mode commands
  • feat(commands): add new select_* commands for interactive node selections
  • feat(config): add higlight; mapping for selection commands
  • fix(config): only setup mappings are removed upon detach
  • fix(highlighting): for normal mode highlighting is like for / search
  • docs(config): reflect changes in readme.md

Breaking changes

  • Mappings aren't not setup by default

Supported nvim

Tested in

NVIM v0.10.0-dev-ac1c23442f
Build type: MinSizeRel
LuaJIT 2.1.0-beta3

Link to demo video:

https://github.com/theHamsta/crazy-node-movement/assets/8136158/ece7532b-1ce3-46e7-9915-d8d676731b26

@hinell hinell changed the title feat(*): overhaul feat(*): overhaul, select_* nodes, etc. Oct 13, 2023
@hinell hinell changed the title feat(*): overhaul, select_* nodes, etc. feat(*): overhaul, new select_* functions Oct 13, 2023
* feat(commands): clearly split normal mode and select mode commands
* feat(commands): add new select_* commands for interactive node selections
* feat(config): add higlight; mapping for selection commands
* fix(config): only setup mappings are removed upon detach
* fix(highlighting): for normal mode highlighting is like for `/` search
* docs(config): reflect changes in readme.md
@theHamsta
Copy link
Owner

theHamsta commented Oct 20, 2023

Hi @hinell ! These all look like sensible changes. Sorry for not seeing them earlier. Do you want to maintain this plugin or a fork of it? I'm not using it that much recently

I'll be happy to review the changes once I find time

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