Injected CSS with insertCSS

[beaufortfrancois]

Using insertCSS instead of executeScript seems to be proper.
Moreover, you may also want to remove "web_accessible_resources": [ "pesticide.min.css" ] from your manifest.

[mrmrs]

Awesome! thanks @beaufortfrancois

[mrmrs]

@paulmolluzzo - merged this down to master. If you can double check everything and update the web store that would be great!

[paulmolluzzo]

@mrmrs This update removes a key feature: Toggling Pesticide.

In the version I wrote, you could show/hide Pesticide by clicking the extension icon. In @beaufortfrancois' update the toggling was removed and instead only turns Pesticide on; you have to refresh the page to remove Pesticide.

As it is now, I'd rather not push this update to the web store since it reduces half the functionality. If there's a better way to write any of the code that doesn't kill the toggling, I'm all for it. I have considered integrating a on/off version of the icon and updating it to make the ability to toggle more apparent.

[beaufortfrancois]

Oh I didn't see that. Sorry.
It is indeed a bad move. executeScript is the way to go then.

[paulmolluzzo]

No worries. I admit that it's tough to read through that ridiculous ternary.