Add option to enable/disable the spell checking by default about fin-spell HOT 9 CLOSED

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

from fin-spell.

Yeah, that sounds very good!

from fin-spell.

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.

from fin-spell.

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.

from fin-spell.

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?

from fin-spell.

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.

from fin-spell.

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?

from fin-spell.

Yes, now it feels very good! Thanks.

from fin-spell.

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

from fin-spell.