goofy icon indicating copy to clipboard operation
goofy copied to clipboard

Add option to disable / change language of spell check

Open tomaskavalek opened this issue 6 years ago • 14 comments

After last update – so I am using version 3.4.1.

Please, allow to disable spellcheck in the app. I am writing a lot of messages in czech language, so every word is with red line below, and every next word slowing down the message box. Allow to choose the language or disabling this spellcheck.

Thanks a lot.

tomaskavalek avatar Feb 13 '19 11:02 tomaskavalek

Probably this piece of code.

tomaskavalek avatar Feb 13 '19 11:02 tomaskavalek

Added a new preferences pane to Goofy v3.4.2 - where I've given the option for the user to disable the spell checker.

I've also added a drop down to select between the two different types of english or use the system locale (might work in some instances?). Will expand to include more languages in future but hopefully the system one should work for now.

jklp avatar Feb 14 '19 09:02 jklp

Ugh that's frustrating ... seems to work on my local machine though changing languages on the github binary doesn't seem to work (disabling spell check does work though)

Re-opening so we can look at in next milestone

jklp avatar Feb 14 '19 10:02 jklp

I have downloaded version 3.4.2 and everything works as I expected.

If I enable spell checker and keep select as System, so app use English spell-checker on my Czech locale. But it's due to missing support from spell-checker – English only.

tomaskavalek avatar Feb 14 '19 10:02 tomaskavalek

If I uncheck spell checker, it's disabled. Good work I think.

tomaskavalek avatar Feb 14 '19 10:02 tomaskavalek

Yeah sorry - was hoping would be a little cleaner but got too excited with the preferences pane I didn’t 100% let it bake before releasing.

More than happy to accept PRs / feedback and now with preferences pane hopefully in good shape to make Goofy a little more configurable.

jklp avatar Feb 14 '19 10:02 jklp

Now using electron-spellchecker to do spell checking which seems a lot more reliable.

jklp avatar Feb 18 '19 07:02 jklp

Still not working. See https://github.com/electron-userland/electron-spellchecker/issues/120. Have you tested it?

tomaskavalek avatar Feb 21 '19 08:02 tomaskavalek

Yep, am located in Australia so I use UK English vs US and spelling errors have been highlighting correctly for me

Unfortunately I don't speak any other languages but I did do some quick testing with French and German via Google Translate which seemed to work - is it not working for you?

By the way you'll need to restart Goofy for the language change to work (I haven't put in hot reloading of any settings yet)

jklp avatar Feb 21 '19 08:02 jklp

Have confirmed, looks like problem upstream with electron-spellchecker - other issues below:

https://github.com/electron-userland/electron-spellchecker/issues/85 https://github.com/electron-userland/electron-spellchecker/issues/61

Have set spell checking to disabled by default as I don't think should remove entire feature if some users will find it helpful - have added doc to let users know the feature isn't 100% working though so hopefully clears things up.

Haven't closed issue but moved out of 3.4.3 - lets revisit when we get some answers from electron-spellchecker (or find another lib which does the job)

jklp avatar Feb 24 '19 00:02 jklp

Hi ! I have a similar problem here !

My system language is English, but I write lot of Czech text too. Altough the spell check is disabled in Goofy preferences window, the app still checks the spelling, and only in English ! That means that all the Czech text is marked as incorrect, and it can't be disabled.

Moreover, changing Goofy spell option several times led to a blank app window and I had to delete all its ~/Library files. The option doesn't work for me at all.

I use Goofy 3.4.3 and MacOS 10.14.4

Palivec avatar Apr 24 '19 15:04 Palivec

@Palivec Hi did you restart Goofy after disabling Spell check in preferences? I just tried disabling on my end and typing in some Czech and no spelling errors are highlighted.

jklp avatar Apr 29 '19 11:04 jklp

Looks like Electron supports spell checking natively now!

https://github.com/electron-userland/electron-spellchecker/issues/85#issuecomment-560006783

jklp avatar Dec 01 '19 12:12 jklp

Just double checked, only supported in Electron 8 (still in beta) - will look into again once Electron 8 is out

jklp avatar Dec 07 '19 07:12 jklp