Nylas-mail: Non-English Spell Checker

[csimonmoi]

Hi
You just released your last update and I do not see any spell checker function on my client. I do not see any red lines under my language (italian) but if I do some mistakes nothing happens and there is no check at all.
Another thing, if I am writing a new email, your client creates new drafts (not a single one, but 2 o 3) and after sending that email, drafts are still there and I have to delete them manually.
Thanks

[mbilker]

OS? Linux has issues that I am working on.

[csimonmoi]

Windows

[mbilker]

What version of Windows (7/8/8.1/10)? Version of N1 (visible under Help, not copyable right now)?

[csimonmoi]

Last one N1 version : 0.3.29

Windows 10

[emorikawa]

@csimonmoi The lack of sqwiggles is likely due to us not detecting a dictionary for your language. On Windows we defer to the native Windows Spell Checking API. That API lets us know what languages are available

You can see what languages you support by looking in:

C:\Users\YourName\AppData\Roaming\Microsoft\Spelling

On my computer I only have an en-US folder. I believe Windows sets these up when you configure your computer.

When I tested on my Windows 10 machine Italian was not in my list of supported languages. We recently changed the behaviour to disable spellcheck if we don't recognize the language. Before this change we would erroneously force an English dictionary on foreign speakers.

As far as the draft issue is concerned. We are addressing an issue related to multiple drafts being created.

[csimonmoi]

Hi thanks for your answer

as you can see in my snapshot attached here Italina laguage is in my machine,
in the folder you mentioned.

But still does not work, in italian but in English too.

as a matter of fact I am writing right now in Nylas client and if I make a
mistake like this shfsdfdhfsdhfsdjh, nothing happens.

waiting for your reply.

[dylntrnr]

After the latest update, I also have no spellchecker. I'm guessing this is because I have spanish and english setup as possible languages. How can I get Nylas to recognize the current language? Thank you.

[mbilker]

@dylntrnr in the Developer Tools (Developer -> Toggle Developer Tools), what is the result of navigator.language and navigator.languages?

Also, try the latest version of N1 (0.3.32-b11131d, link for OS X) and see what the result is in the new version.

[csimonmoi]

Hi

even after last update (this morning) I have the same issue about spellchecker in Italian and in English (not at all any spellchecker). As you already know from my snapshot I sent to you, my OS is Windows 10.

[mbilker]

Could you please run in the developer tools and report the outputs of each command other than the first:

var sp = new (require('spellchecker').Spellchecker)(); // ignore the output of this command
sp.setDictionary(navigator.language);
sp.isMisspelled('gnocchi');
sp.isMisspelled('gnochi');

[mbilker]

Also, please give me the output of navigator.language.

[csimonmoi]

Hi

I am not a developer or whatsoever, I am only an end user. If you can give me a little more help to lauch Developer Tool (Preferences > Toggle Developer Tools > and then I see only a lot of stuff out there) commands.

[mbilker]

Yes it is Developer -> Toggle Developer Tools. In there you should paste:

var sp = new (require('spellchecker').Spellchecker)();
sp.setDictionary(navigator.language);
sp.isMisspelled('gnocchi');
sp.isMisspelled('gnochi');

Then take a screenshot. It should not contain personal information, but if you are concerned, take a screenshot starting with var sp and ending with ');.

[csimonmoi]

Hi there

sorry but Developer tools has MANY tools:

on top: Elements, Network, Sources, etc.

I thought you intended "Console" on bottom of the tool.

I pasted your commands in Console and the only word that popped up is "true".

Let me know if it is that what you intentend or pleas be more specific.

Thanks

[dylntrnr]

@mbilker After updating to the latest version english spell check is working. thank you

attached is the output in console

[csimonmoi]

Attached below what I think you think I had to do.

On Dec 16 2015, at 4:41 pm, Matt Bilker <[email protected]>
wrote:

Yes it is Developer -> Toggle Developer Tools. In there you should paste:

var sp = new (require('spellchecker').Spellchecker)();  
sp.setDictionary(navigator.language);  
sp.isMisspelled('gnocchi');  
sp.isMisspelled('gnochi');



    Then take a screenshot. It should not contain personal information, but
if you are concerned, take a screenshot starting with `var sp` and ending with
`');`.




—  
Reply to this email directly or [view it on GitHub](https://github.com/nylas/N
1/issues/670#issuecomment-165148073).![](https://github.com/notifications/beac
on/APTadqRxmWB98wj_Z-n4h-Qmn0rccCvMks5pQX2RgaJpZM4GzjgU.gif)

[mbilker]

@csimonmoi Thank You! That is telling me the backend of the spell checker is working properly. In the Console tab at the top of the Developer Tools, could you type $n.NylasSpellchecker.languageAvailable

[mbilker]

@csimonmoi thinking about it a little more. I don't know which command returned true so please rerun the commonds in the Console tab at the top of the Developer Tools. Thanks!

[jstejada]

hey @csimonmoi , is the spellchecker still not working for you?

[csimonmoi]

I tried what you suggest typing in the console:

$n.NylasSpellchecker.languageAvailable

The result is "false".

No spell checker still, non in English, no in Italian.

Windows 10.

[kios]

I have the same problem in Windows 7 64-bit Spanish. with version 0.3.29, everything worked properly, sorry for my English, I only speak Spanish

[csimonmoi]

Hi team
My system is Win 10 and sent you many request about spell checker which does
not work in Italina nor in English.

There are other issues in Win10:

When I open FOR THE FIRST TIME in the morning (after a shuddown) your program
the main window is not well positioned on the screen, and I have to click to
minimize and then maximize again.

Same story about column width. If you close and reopen the program everything
is ok. But if you shutdown the computer and reopen the program after you boot
the computer up, then the width is not arranged as you chose last time.

Thanks

[bilby91]

I could never make the Spanish spell checker work on OS X. I have Spanish as a my secondary language in OS X preferences.

Running N1 0.3.45-3b5880e and OS X 10.11.1 (15B42).

Let me know if you need any debugging information.

[kakubei]

Same here on a Mac running Yosemite. I have the same result as dylntrnr:

navigator.language
"en-US"
navigator.languages
[]

My OS is set to English, but I also have Spanish loaded under Language & Region. Nylas version is: Version 0.4.5-b0ec335 (0.4.5-b0ec335)

Any ideas?

[simonbcn]

I have same problem on Arch Linux with Spanish.

[mbilker]

@simonbcn Only English is supported at the moment on all Linux distributions

[mbilker]

I determined that Italian is not a default supported language on Windows and requires a language pack. It is possible to include the Hunspell spellchecker on Windows, but it is not the best solution and may not even work.

[mbilker]

Drawing from Slack's example with node-spellchecker, they force Hunspell on Linux and use the system dictionaries by default, which can be a large number of languages. On Mac OS X, they use the OS provided spellchecker because it is much better than Hunspell. On Windows 8 and above, they use the OS provided spellchecker even if it is missing dictionaries. On Windows 7 and below, they use Hunspell and download the dictionaries from
http://slack-ssb-updates.s3.amazonaws.com/dictionaries
and checks
http://slack-ssb-updates.s3.amazonaws.com/dictionaries/RELEASES.

[csimonmoi]

Sorry but your spell checker does NOT work not only in Italian but in any
other language even English.

As now ... slfksfskdlfjdslfsl .... there is no correction at all.

Thanks anyway

[mbilker]

It appears Slack doesn't check if the language exists on Windows or Mac OS X, because those API's are not implemented in node-spellchecker.

EDIT: Correction, they are implemented, but from the reports here that method is unreliable.

[jstejada]

merging into #1392