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

Some math expressions are spelled in LaTeX whereas they should not be #35

Closed
bartosz-antosik opened this issue Aug 5, 2017 · 3 comments

Comments

@bartosz-antosik
Copy link
Owner

James Yu suggested in one of comments to issue #225 that Spell Right spells math expressions like:

\forall(i,j)\notin\mathcal{E}

where i gets underlined as spelled incorrectly (suggesting capital).

They should not be spelled.

@bartosz-antosik
Copy link
Owner Author

Correct me if I am wrong, but it seems to me that this expression would get spelled only outside of math braces ($ and $ etc.), where it should not exist in this form. I assumed that only \command{...}... and \command[..]... should not be spelled in normal document context and nothing inside math braces.

@James-Yu
Copy link

James-Yu commented Aug 5, 2017

In fact, in LaTeX it is very common to have math equations in the following form:

\begin{align}
\some_{equations}^{here}
\end{align}

The snippet I provided was also surrounded by the environment. The way cSpell handles this problem is simply ignore any words with three or less letters, which I found to be surprisingly effective and not causing many false negatives. Just for your reference.

@bartosz-antosik
Copy link
Owner Author

You are right. The expression will be eliminated by modified regexp. Next release will have it.

About spelling starting from some length of the word, e.g. 4 characters. I am very strong against about spelling starting from some length! This was in fact my objection against most of the spell checkers I have examined before starting mine on the beginning of this year. It makes impossible to spell a lot of short words, shortcuts and abbreviations (I'm, etc., i.e. etc., etc.) As I was stressing my intention was to produce a precise speller thus a lot of tweaking in the parsers. Like the thread above...

bartosz-antosik pushed a commit that referenced this issue Aug 6, 2017
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

2 participants