Skip to content

0.1.2 - adds options V2 page, allows whitelisting #18

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zachhardesty7
Copy link

After using this extension for a number of years, I've run across a
handful of sites that use .gifs as button, etc. This extension wrongfully
targets them and often accidentally breaks the formatting and
functionality of the website.

My fix simply implements a whitelist of sites that disable the content
scripts through programmatic injection. Within background.js, my edits
check a parsed version of the contents of the whitelist versus the current
domain. If there are matches, the extension never executes any changes on
the page.

After using this extension for a number of years, I've run across a
handful of sites that use .gifs as button, etc. This extension wrongfully
targets them and often accidentally breaks the formatting and
functionality of the website.

My fix simply implements a whitelist of sites that disable the content
scripts through programmatic injection. Within background.js, my edits
check a parsed version of the contents of the whitelist versus the current
domain. If there are matches, the extension never executes any changes on
the page.

 - Contribution by Zach Hardesty: http://zachhardesty.com
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

Successfully merging this pull request may close these issues.

1 participant