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 whacky macOS logic in need of ❤? #127

Open
bengotow opened this issue Nov 4, 2018 · 0 comments
Open

Some whacky macOS logic in need of ❤? #127

bengotow opened this issue Nov 4, 2018 · 0 comments

Comments

@bengotow
Copy link
Contributor

bengotow commented Nov 4, 2018

Hey folks - I've been investigating reports that this module doesn't auto-detect the spellcheck language on macOS, which appears to be true based on some straightforward tests.

I think that the code in electron-spellchecker has been through a lot of iteration and improvement, but there's some whacky stuff going on around automatic language detection. Like this if statement - this else-if scenario will never fire with it ordered after the first case. I'm not sure I understand that comment, but it seems like this isn't quite right.

image

There's also an LRU cache which electron-spellcheck maintains on macOS, but if the NSSpellchecker is switching languages and the cache is never busted, things will just continue to be reported as misspelled:

image

@bengotow bengotow changed the title Some whacky macOS logic... Some whacky macOS logic in need of ❤? Nov 4, 2018
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

1 participant