-
Notifications
You must be signed in to change notification settings - Fork 434
--disable-remote-fonts makes uBlock unusable #1181
Comments
I didn't think the FontAwesome fonts were "remote"; anyway, I think PNGs would be better because of alpha blending. |
@lewisje at one level, yes, but the sacrifice in interoperability across pixel densities is quite significant. Still, it'd be feasible to use SVGs instead. At that point, though, that's pretty much as good as just packaging and shipping FontAwesome with uBlock so it's all local. |
I briefly looked at the code and didn't see where FontAwesome was called; I originally thought it was local and was surprised that it's not. |
Upon further investigation, FontAwesome isn't actually "remote" currently in uBlock. Chrome's |
Found this, https://icomoon.io/ You pick what font library you want ( Font Awesome ) and select just the icons you want to use. Then you can generate SVG or a subset of the font. Below is the 7 icons generate into svg that I could find uBlock using. Might be missing some. I just did a search for "class fa" on github. Edit: |
@my-password-is-password yep; thanks! Very neat. I've integrated an implementation of that into my local clone of the repo, and will be pushing it very soon (just need to make sure I didn't break anything obvious). Side note: there are problems with the FontAwesome icons in SVG format (misalignment and incorrect sizes) — mainly because FontAwesome doesn't really provide the glyphs in independent SVGs. So, I went with slightly different icons for some of them (the differences are visual and minor). |
Hi @octoploid! Good news! uBlock now uses a set of SVGs it ships with for all the icons. The icons are much lighter, and, of course, it works even if you have web fonts disabled. There were multiple commits of relevance in the range [a947537...53768af). |
Thank you Chris. |
Many thanks. |
Could you please use gifs instead?
Thanks
The text was updated successfully, but these errors were encountered: