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 "Disable AMP" userscript to the list of default userscripts #2815

Closed
ameshkov opened this issue Jun 17, 2019 · 5 comments

Comments

@ameshkov
Copy link
Member

commented Jun 17, 2019

We should first develop this userscript :)

@TPS

This comment has been minimized.

Copy link
Contributor

commented Jun 20, 2019

Why, though? Not only do I find adblocking AMP pages more efficient, y'all could use 1 of the many extant 1s. (E.g., https://github.com/bentasker/RemoveAMP comes up first from Google search.)

N.B.: To implement #2723?

@ameshkov

This comment has been minimized.

Copy link
Member Author

commented Jun 20, 2019

There are tons of people who hate AMP so why not? It'll be disabled by default anyway.

@TPS

This comment has been minimized.

Copy link
Contributor

commented Jun 20, 2019

Nono, I mean, why reinvent this wheel? Why not just link, e.g., the above userscript?

@ameshkov

This comment has been minimized.

Copy link
Member Author

commented Jun 20, 2019

We'd like to avoid including any third-party userscript by default. Userscripts are simply too powerful and dangerous in the wrong hands.

@ameshkov

This comment has been minimized.

Copy link
Member Author

commented Jun 28, 2019

@artemiv4nov the first very-very simple version:
https://userscripts.adtidy.org/release/disable-amp/1.0/disable-amp.user.js

(Does not work on Google News yet)

Please check that it works properly on Android.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.