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

Why is lowercase "th" not flagged as an error, e.g. "look at th typo" #935

Open
justingrant opened this issue Feb 22, 2022 · 2 comments
Open

Comments

@justingrant
Copy link

I just had to re-do a documentation PR because CSpell didn't catch the word "th" as being a typo in a markdown file. I understand why "Th" (for the element Thorium) is accepted, but why was lowercase "th" accepted? Are all capitalized words' lowercase variants automatically accepted too? Or is it accepted because markdown can accept HTML elements and the CSpell markdown parser isn't sophisticated enough to distinguish between element names (where th is OK) and regular text where it's always a typo?

FWIW, http://app.aspell.net/lookup?dict=en_US;words=th shows that the lowercase "th" isn't present.

@nschonni
Copy link
Collaborator

https://github.com/streetsidesoftware/cspell/tree/main/packages/cspell#cspelljson-sections minWordLength default is 4, so it doesn't check words shorter than 4 characters unless you override it

@revelt
Copy link
Contributor

revelt commented Jul 20, 2022

There are two cases, /th tag/i (references to HTML tag) and misspelt "the".
I think if possible, we should try to recognise both, out of the box, on default settings.

For comparison, Grammarly is on the opposite extreme:
Screenshot 2022-07-20 at 09 21 55

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

No branches or pull requests

3 participants