Skip to content

Conversation

@iagorrr
Copy link

@iagorrr iagorrr commented Sep 6, 2025

feat: add spell checker for English

    - The plugin `spellwarn` was added, and set to support English.
    - The flake.nix and flake.lock was also update to include the new plugin.
    - A file size limit as added to avoid lag when dealing with large files
    (larger than 80 lines)

@iagorrr iagorrr changed the title feat: add spell checker for both English feat: add spell checker for English Sep 6, 2025
@iagorrr iagorrr marked this pull request as draft September 6, 2025 14:40
- The plugin `spellwarn` was added, and set to support English.
- The flake.nix and flake.lock was also update to include the new plugin.
- A file size limit as added to avoid lag when dealing with large files
(larger than 80 lines)
@iagorrr iagorrr force-pushed the feat-spell-warn-upstream branch from 04b4820 to 5575c35 Compare September 6, 2025 14:52
@iagorrr iagorrr marked this pull request as ready for review September 6, 2025 14:56
@iagorrr iagorrr marked this pull request as draft September 6, 2025 15:46
@iagorrr iagorrr marked this pull request as ready for review September 6, 2025 20:03
@SystematicError SystematicError force-pushed the master branch 2 times, most recently from 79ee1a2 to 95aa559 Compare September 8, 2025 11:21
@SystematicError
Copy link
Owner

Thanks for the PR! I haven't really tried out spellchecking code, so I'm not too sure if this is a feature that should be there by default, I'll test it out and see how it goes.

PS: I've added some documentation regarding custom user configuration in the README, you can define your own plugins and configs through that.

@iagorrr
Copy link
Author

iagorrr commented Sep 8, 2025

... so I'm not too sure if this is a feature that should be there by default, I'll test it out and see how it goes.

The built-in Neovim spell checker works very well; however, I find that the SpellWarn plugin becomes too laggy with larger files or too many errors, which makes for a poor user experience. Maybe only the built-in should be default, but i'll put some extra time to check if i made anything wrong.

PS: Thanks for the documentation :))

@iagorrr
Copy link
Author

iagorrr commented Sep 11, 2025

... but i'll put some extra time to check if i made anything wrong.

Still, I couldn’t reach a usable state — the plugin made everything feel laggy.
Meanwhile, the built-in spell checker works exceptionally well out of the box; perhaps it should be enabled by default?

Would literally be just this:

vim.opt.spell = true
vim.opt.spelllang = { "en_us" }

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