Skip to content
This repository has been archived by the owner on Feb 23, 2021. It is now read-only.

Add option to disable / change language of spell check #442

Open
tomaskavalek opened this issue Feb 13, 2019 · 14 comments
Open

Add option to disable / change language of spell check #442

tomaskavalek opened this issue Feb 13, 2019 · 14 comments
Milestone

Comments

@tomaskavalek
Copy link
Contributor

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
Copy link
Contributor Author

Probably this piece of code.

@jklp jklp added this to the 3.4.2 milestone Feb 13, 2019
@jklp jklp changed the title Spellcheck slowing down the app Add option to disable / change language of spell check Feb 13, 2019
@jklp
Copy link
Collaborator

jklp commented Feb 14, 2019

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 jklp closed this as completed Feb 14, 2019
@jklp
Copy link
Collaborator

jklp commented Feb 14, 2019

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 jklp reopened this Feb 14, 2019
@jklp jklp modified the milestones: 3.4.2, 3.4.3 Feb 14, 2019
@tomaskavalek
Copy link
Contributor Author

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
Copy link
Contributor Author

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

@jklp
Copy link
Collaborator

jklp commented Feb 14, 2019

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
Copy link
Collaborator

jklp commented Feb 18, 2019

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

@tomaskavalek
Copy link
Contributor Author

Still not working. See electron-userland/electron-spellchecker#120. Have you tested it?

@jklp
Copy link
Collaborator

jklp commented Feb 21, 2019

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 jklp reopened this Feb 21, 2019
@jklp
Copy link
Collaborator

jklp commented Feb 24, 2019

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

electron-userland/electron-spellchecker#85
electron-userland/electron-spellchecker#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 jklp removed this from the 3.4.3 milestone Feb 24, 2019
@Palivec
Copy link

Palivec commented Apr 24, 2019

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

@jklp
Copy link
Collaborator

jklp commented Apr 29, 2019

@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 jklp added this to the 3.4.8 milestone Dec 1, 2019
@jklp
Copy link
Collaborator

jklp commented Dec 1, 2019

Looks like Electron supports spell checking natively now!

electron-userland/electron-spellchecker#85 (comment)

@jklp
Copy link
Collaborator

jklp commented Dec 7, 2019

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

@jklp jklp modified the milestones: 3.5.2, 3.5.3, 3.5.4 Dec 7, 2019
@jklp jklp removed this from the 3.5.4 milestone Feb 23, 2020
@jklp jklp added this to the 3.5.5 milestone Feb 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants