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

Add option to enable/disable the spell checking by default #7

Closed
wapsi opened this issue Jan 20, 2021 · 9 comments
Closed

Add option to enable/disable the spell checking by default #7

wapsi opened this issue Jan 20, 2021 · 9 comments

Comments

@wapsi
Copy link

wapsi commented Jan 20, 2021

Please add an option to enable (or disable) the spell checking by default. Now I have to click on the Finnish flag icon in the toolbar and click "Päälle". It would be really nice if there was an option to enable or disable the spell checking by default.

Or even better: If there was a possiblity to add different sites (by using regex for example) where the spell checking is enabled or disabled by default, depending of the setting I suggested above.

@fluks
Copy link
Owner

fluks commented Jan 21, 2021

What do you think if we have an option for always enabled and a blacklist or always disabled and a whitelist? So having both.

@wapsi
Copy link
Author

wapsi commented Jan 21, 2021

Yeah, that sounds very good!

@fluks
Copy link
Owner

fluks commented Jan 22, 2021

Okay, I did this. Now you need to refresh the page if you change the setting. Is it a problem? It can't be changed with the current implementation. I used contentScripts API, it made things easy.

@fluks
Copy link
Owner

fluks commented Jan 22, 2021

Were you able to run the add-on in the development environment? I could push the commits and you could test them there.

I pushed enabled-disabled branch for testing.

@wapsi
Copy link
Author

wapsi commented Jan 23, 2021

Yes, I was able to run and build the extension by using the sources I grabbed from here (git, enabled-disabled branch).

Now you need to refresh the page if you change the setting. Is it a problem?

To be honest, I think this is a problem because if I want to turn the spell checking on/off temporarily from the flag icon it does not turn on/off it immediately: so the worst scenario is that I have to refresh the page and I will lose all the text I've written...

Maybe I revert what I said earlier: Is it possible to use the Always Disabled / Always Enabled feature as the permanent setting and in addition to that there is possiblity to turn off/on the spell checking temporarily from the flag icon (and maybe from the context menu in the future #8) and it disables/enables the spell checking immediately for the active tab only?

@fluks
Copy link
Owner

fluks commented Jan 23, 2021

Maybe I revert what I said earlier: Is it possible to use the Always Disabled / Always Enabled feature as the permanent setting and in addition to that there is possiblity to turn off/on the spell checking temporarily from the flag icon (and maybe from the context menu in the future #8) and it disables/enables the spell checking immediately for the active tab only?

Yes, I can do that, I think.

@fluks
Copy link
Owner

fluks commented Jan 24, 2021

In the latest commit in the development branch, the functionality is now that the options control spell checking globally and you can temporarily enable or disable spell checking in the active tab. Refreshing the tab restores the global setting. Does it now work as you would like?

@wapsi
Copy link
Author

wapsi commented Jan 26, 2021

Yes, now it feels very good! Thanks.

@wapsi
Copy link
Author

wapsi commented Feb 3, 2021

I think this issue / feature request can be closed if you want to? I can confirm that this functionality is added and tested, thanks!

@fluks fluks closed this as completed Feb 3, 2021
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