search

ankit / stylebot

Change the appearance of the web instantly
https://stylebot.dev/
MIT License
1.4k stars 208 forks source link

Why stylebot add index.css to all website? #571

Closed k79e closed 4 years ago

k79e commented 4 years ago

It updated to v3 today and I find that. Is it normal?

It added a index.css (chrome-extension://oiaejidbmkiecgbjeifoejpgmdaleoha/editor/index.css) and https://fonts.googleapis.com/css2?family=Inter:wght@300;400;500&display=swap

Even if I disable all rules it still add these css automatically.

ankit commented 4 years ago

@k79e yes, the css is added for the stylebot editor. it should not conflict with any other elements on the page since it is injected into a shadow DOM. are you noticing any side effects in terms of performance?

k79e commented 4 years ago

Is this necessary? It even load two css when I only open a picture. And this second css is loading some woff fonts. Loading these thing on other website will make the load time become wrong.

I think even if there don't exist any problem it still should not add these css to other website. Because you make this behavior out of control from user. It will be horrible.

WORMSS commented 4 years ago

Can we please remove this? I am doing load timing for websites I have a simple display: none on an element on. I don't need font loading.

ankit commented 4 years ago

@k79e thanks for bringing this up. I will have a fix in the next release.

WORMSS commented 4 years ago

Typical, the day after I uninstalled the extension, you fix it.