Skip to content

chores: rearrange wm keymaps#1363

Merged
charliie-dev merged 1 commit intoayamir:mainfrom
AngelontheRoad:main
Nov 4, 2024
Merged

chores: rearrange wm keymaps#1363
charliie-dev merged 1 commit intoayamir:mainfrom
AngelontheRoad:main

Conversation

@AngelontheRoad
Copy link
Contributor

I put wm keymaps into keymap/ui.lua and divided them into builtin_map and plug_map. It's strightfoward and maybe you could like it.

@charliie-dev
Copy link
Collaborator

charliie-dev commented Nov 4, 2024

It make sense to me to move those windows core keymaps to keymap/ui.lua, but it would be even better if you move the remaining edit core keymaps to keymap/editor.lua.

Don't forgot to remove this line, update tutor/dots.tutor and update wiki!

Copy link
Owner

@ayamir ayamir left a comment

Choose a reason for hiding this comment

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

LGTM

@AngelontheRoad
Copy link
Contributor Author

It make sense to me to move those windows core keymaps to keymap/ui.lua, but it would be even better if you move the remaining edit core keymaps to keymap/editor.lua.

Don't forgot to remove this line, update tutor/dots.tutor and update wiki!

Right, I've done.

Copy link
Collaborator

@charliie-dev charliie-dev left a comment

Choose a reason for hiding this comment

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

LGTM, also https://github.com/ayamir/nvimdots/wiki/Keybindings is updated.
ready to merge.

@charliie-dev charliie-dev merged commit f475d96 into ayamir:main Nov 4, 2024
Deg2v pushed a commit to Deg2v/nvimdots that referenced this pull request Jan 16, 2025
Jint-lzxy added a commit that referenced this pull request Jan 18, 2025
This is basically a follow-up for #1363! I combined builtin & plugin keymaps,
cleaned up some comments (lol "suckless"), and made `ui.lua` export-ready

Signed-off-by: Jint-lzxy <50296129+Jint-lzxy@users.noreply.github.com>
ayamir pushed a commit that referenced this pull request Jan 19, 2025
This is basically a follow-up for #1363! I combined builtin & plugin keymaps,
cleaned up some comments (lol "suckless"), and made `ui.lua` export-ready

Signed-off-by: Jint-lzxy <50296129+Jint-lzxy@users.noreply.github.com>
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.

3 participants