Skip to content

feat: add render-markdown.nvim for md workflow#1336

Merged
ayamir merged 4 commits intomainfrom
feat/render-markdown
Aug 9, 2024
Merged

feat: add render-markdown.nvim for md workflow#1336
ayamir merged 4 commits intomainfrom
feat/render-markdown

Conversation

@charliie-dev
Copy link
Collaborator

charliie-dev and others added 4 commits August 9, 2024 10:59
Signed-off-by: Jint-lzxy <50296129+Jint-lzxy@users.noreply.github.com>
Signed-off-by: Jint-lzxy <50296129+Jint-lzxy@users.noreply.github.com>
Signed-off-by: Jint-lzxy <50296129+Jint-lzxy@users.noreply.github.com>
Copy link
Collaborator

@Jint-lzxy Jint-lzxy left a comment

Choose a reason for hiding this comment

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

LGTM 🥳 I made a few tweaks in the latest commits:

  • Mapped <F1> to toggle the plugin, keeping it consistent with <F12> and letting users quickly turn it off if needed.
  • Disabled the plugin for files over 2 MiB based on my tests (files this large caused noticeable lag). This also aligns with bigfile's config.
  • Added nvim-web-devicons as a dependency, as per the README.

@ayamir ayamir merged commit ccf0464 into main Aug 9, 2024
@ayamir ayamir deleted the feat/render-markdown branch August 9, 2024 06:35
This was referenced Sep 23, 2024
Deg2v pushed a commit to Deg2v/nvimdots that referenced this pull request Jan 16, 2025
* feat: add `render-markdown.nvim` for md workflow

* feat: `<F1>` toggles the plugin

Signed-off-by: Jint-lzxy <50296129+Jint-lzxy@users.noreply.github.com>

* fix: add `nvim-web-devicons` as a dependency

Signed-off-by: Jint-lzxy <50296129+Jint-lzxy@users.noreply.github.com>

* perf: disable for files larger than 2 MiB

Signed-off-by: Jint-lzxy <50296129+Jint-lzxy@users.noreply.github.com>

---------

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