Skip to content

docs(nvim): add documentation for neovim#3

Open
Prakhar1808 wants to merge 7 commits into
LilSuperUser:mainfrom
Prakhar1808:docs/nvim-documentation
Open

docs(nvim): add documentation for neovim#3
Prakhar1808 wants to merge 7 commits into
LilSuperUser:mainfrom
Prakhar1808:docs/nvim-documentation

Conversation

@Prakhar1808

@Prakhar1808 Prakhar1808 commented Apr 11, 2026

Copy link
Copy Markdown
Contributor

Overview

this PR adds documentation for the nvim dotfiles (config)
majorly all keymaps needed to use the nvim config

Screenshot (Preview)

image

Note For Maintainers

do tell if any changes are to be made or any specific keymaps need to be added or removed

@Prakhar1808 Prakhar1808 marked this pull request as ready for review April 14, 2026 14:28

@LilSuperUser LilSuperUser left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

We need a few changes in the keymap documentation:

  • I noticed that you have missed some keymaps that exists in the config. Please add them.
  • We should also add a Mode column (to be verbose about what mode the user has to be in to use the keymap
  • I noticed one duplicate entry under the General Section, 1st and the 3rd keymaps are same. Adding a column for Mode would fix it.

Lastly, the current structure makes it hard to navigate cause the user would have to scroll a lot to find the desired keymap.

Maybe we should organize it as a webpage or adding an index/table of contents at the top. The main idea is that it should look like a cheatsheet.

Also, I’ve pushed some changes to main, please pull and rebase your branch on top of it.

@Prakhar1808

Copy link
Copy Markdown
Contributor Author

thanks for the input, will do!

@Prakhar1808

Copy link
Copy Markdown
Contributor Author

additions after your comment:

  • Mode Column (for visual, insert and normal mode)
  • Remaining Keymaps
  • Table of Contents (better navigation)

@LilSuperUser LilSuperUser left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please check that all the links in the table of contents are working correctly.

Also, I think it would be helpful to add a brief one-line description for each entry so users can quickly understand what functionality or section the link points to.

Comment thread README.md
- [Splits](#splits)
- [Snacks](#snacks)
- [Linting](#linting)
- [Mini](#mini)

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

The link to Mini section is wrong

Comment thread README.md
Comment on lines +29 to +30
- [Clasp](#clasp)
- [Oil](#oil)

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

The link to Clasp and Oil section is wrong

Comment thread README.md
Comment on lines +216 to +228
### Multi-Cursor
| Function | Keymap | Mode |
| -------------- | --------------- | ----- |
| Line Skip Cursor (+1) | `<space>j` | n |
| Line Skip Cursor (-1) | `<space>k` | n |

---

### Color-Picker
| Function | Keymap | Mode |
| -------------- | --------------- | ----- |
| Toggle color highlighter | `<leader>cc` | n |

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Some keymaps are still missing in the documentation.

I suggest taking checking out each plugin's config and then documenting the keymaps. (Not all of the keymaps are shown by which-key plugin

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'll make sure to make the necessary changes

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