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

Spell check for source editor, notebooks, etc. #1485

Open
jmcphers opened this issue Oct 3, 2023 · 6 comments
Open

Spell check for source editor, notebooks, etc. #1485

jmcphers opened this issue Oct 3, 2023 · 6 comments
Labels
area: core Issues related to Core category.
Milestone

Comments

@jmcphers
Copy link
Collaborator

jmcphers commented Oct 3, 2023

VS Code doesn't have any global spell checking service, and does not have built-in spell checking for most of its components. This is a long-standing issue that has created a lot of third-party spelling plugins.

Because we want Positron to have a pleasant scientific authoring experience out of the box, without installing plugins, we should bundle a spellcheck extension, or add a spellcheck service as part of the core or editor system.

Some notes on existing art:

I generally think it would be preferable to build our own spell check service as opposed to depending on a third party plugin.

Here's spellcheck working against a visual mode doc with Quarto:

image

@EmilHvitfeldt
Copy link

It would be nice if it also worked in the various chunk options like fig-cap and fig-alt

@chendaniely
Copy link

is it possible to make the spell check fix suggestions show up on right-click?
it's one of the biggest annoyances I have with all the spell checkers is hovering over and clicking the right thing. sometimes the window disappers on me before I can click on the suggestions box in regular vscode

@chendaniely
Copy link

If we do have a spell checker that ships with positron, we might want to also curate a list of words for a custom dictonary too.

e.g., tabset for quarto ::: {.panel-tabset}

not sure if there's a better place to put in words for custom dictonary things.

@jmcphers
Copy link
Collaborator Author

We already have such a list, curated for RStudio:

https://github.com/rstudio/rstudio/blob/main/src/gwt/src/org/rstudio/studio/client/common/spelling/domain_specific_words.csv

Although for your specific example, I think that it'd be better for us just to recognize ::: as a fence line that doesn't need spell checking.

@thomasp85
Copy link

Also, spell checking in roxygen comments (maybe comments in general) - I'm sure it is on the radar and part of the general spell-checking effort, so just adding it here for completion

@jmcphers
Copy link
Collaborator Author

Unsurprisingly, this has already been requested by a beta user: https://github.com/posit-dev/positron-beta/issues/121

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: core Issues related to Core category.
Projects
None yet
Development

No branches or pull requests

5 participants