classilla / tenfourfox

Mozilla for Power Macintosh.
http://www.tenfourfox.com/
Other
270 stars 38 forks source link

Update font blacklist #566

Open classilla opened 4 years ago

classilla commented 4 years ago

From https://tenfourfox.tenderapp.com/discussions/problems/8844-waste-management-have-broken-themselves-on-tenfourfox-with-their-new-and-improved-yeah-right-website

classilla commented 4 years ago

Tested working

chris-chtrusch commented 4 years ago

https://www.kulturstiftung-des-bundes.de/typo3conf/ext/base_ksb/Resources/Public/38c1bdeb69b2cae2f59fae38f127aa6d.woff2

https://www.kulturstiftung-des-bundes.de/typo3conf/ext/base_ksb/Resources/Public/9437aea92626957f9d7fd0e5f548b689.woff2

classilla commented 4 years ago

Okay, will add for FPR17.

chris-chtrusch commented 4 years ago

Is it possible to blacklist https://www.kulturstiftung-des-bundes.de/typo3conf/ext/base_ksb/Resources/Public/*.woff ? They seem to switch this part of the URL every once in a while.

classilla commented 4 years ago

Yes, I'll change the entry. It's designed so that it can match on partial URLs.

chris-chtrusch commented 4 years ago

https://cdn.trustpilot.net/brand-assets/2.1.0/fonts/trustpilot-default-font-regular.woff2 https://cdn.trustpilot.net/brand-assets/2.1.0/fonts/trustpilot-default-font-medium.woff2 https://cdn.trustpilot.net/brand-assets/2.1.0/fonts/trustpilot-default-font-bold.woff2

classilla commented 4 years ago

I'll pick this up for FPR21 since FPR20 is already halfway through build and certify.

chris-chtrusch commented 4 years ago

Trustpilot still has these font issues.

classilla commented 4 years ago

Damn it, this slipped. I'll get for final.

classilla commented 4 years ago

New sets landed, tested working.

chris-chtrusch commented 4 years ago

https://hartzfacts.de/google-fonts/s/notoseriftc/v7/XLY9IZb5bJNDGYxLBibeHZ0Bvr8vXX5MTw.woff2

https://hartzfacts.de/google-fonts/s/notoseriftc/v7/XLY9IZb5bJNDGYxLBibeHZ0BvtssXX5MTw.woff2

https://hartzfacts.de/google-fonts/s/notoseriftc/v7/XLYgIZb5bJNDGYxLBibeHZ0BtnAOSA.woff2

https://hartzfacts.de/google-fonts/s/notoseriftc/v7/XLY9IZb5bJNDGYxLBibeHZ0BvoMtXX5MTw.woff2

https://hartzfacts.de/google-fonts/s/notoseriftc/v7/XLY9IZb5bJNDGYxLBibeHZ0BvvMpXX5MTw.woff2

classilla commented 4 years ago

Ack'ed for FPR25

chris-chtrusch commented 2 years ago

I found two more fonts to add to the font list. For my own build I did this at gfxPlatformMac.cpp:

    EXACT_URL("https://som.yale.edu/themes/custom/som/fonts/neuehaasunica/NeueHaasUnicaBlackItalic/italic_normal_subset1.woff2");

    EXACT_URL("https://som.yale.edu/themes/custom/som/fonts/neuehaasunica/NeueHaasUnicaBlack/normal_normal_subset1.woff2"); 

This worked, but EXACT_URL is probably the least generic/desirable place to add it.

classilla commented 2 years ago

How about

HTTP_OR_HTTPS_SUBDIR("som.yale.edu","/themes/custom/som/fonts/neuehaasunica/NeueHaasUnicaBlack");

Or does that block too many things?

I'm getting together another patch drop, so I'll include this blocklist update.

chris-chtrusch commented 2 years ago

That should work as well. This being my first try to manipulate the font list I simply didn't know how to use the more generic methods.

Should I keep posting fonts here or is there a way to contribute directly, i.e. submit changes for review? The Unofficial TenFourFox Development Toolkit I use to build the browser doesn't have full Git functionality, I can merge my local changes when I re-download the source, but cannot "push" my changes upwards.

classilla commented 2 years ago

Officially the only way Github does it is to set up a full pull request, which isn't ideal. Oh well.

Short of manually posting URLs, one other option is to do a git diff with your changes and post the diff. I'm okay with either.

classilla commented 2 years ago

You may want to remove your local change before pulling or you'll get a merge conflict, but this is now upstreamed.

chris-chtrusch commented 1 year ago

https://www.washingtonpost.com/wp-stat/assets/fonts/PostoniWide-Regular.woff2 https://www.washingtonpost.com/wp-stat/assets/fonts/PostoniWide-Bold.woff2

chris-chtrusch commented 1 year ago

Major liberal German Newpaper did a site re-design without consulting TenFourFox.

https://assets.tagesspiegel.de/fonts/Abril_Text_Regular-basic.c371b6fb87298eb4990a97124a47e7e4.woff2

https://assets.tagesspiegel.de/fonts/Abril_Text_Regular-extended.413ee6f014cfa7c05f2979ccb6498cee.woff2

https://assets.tagesspiegel.de/fonts/Abril_Text_Italic-basic.6c5932e5019579d6c76d1822e5aacc4d.woff2

https://assets.tagesspiegel.de/fonts/Abril_Text_Bold-extended.7e1a83a609780222181670109982a7a2.woff2

https://assets.tagesspiegel.de/fonts/Abril_Text_Bold-basic.ed72f8ea0597f63d7cc1a47ae7bb3acd.woff2

https://assets.tagesspiegel.de/fonts/Abril_Text_BoldItalic-basic.1cd71e407d313b5db30f65b5a9bc639f.woff2

https://assets.tagesspiegel.de/fonts/franklingothic-book-extended.b0f1fe099d20b0de899ed94f6292ba63.woff2

https://assets.tagesspiegel.de/fonts/franklingothic-book-other.6c8ff800d4fec0ff804a756fc400d2b5.woff2

https://assets.tagesspiegel.de/fonts/franklingothic-book-basic.c62561d04e5eb023ca3e2fde512605c0.woff2

https://assets.tagesspiegel.de/fonts/franklingothic-med-extended.e84bf4c79c0055f692e535db121cff4a.woff2

https://assets.tagesspiegel.de/fonts/franklingothic-med-basic.4262342c15da29dbdffaff795d31ad13.woff2

classilla commented 1 year ago

Noted, thank you. Probably will do a regular update around now too.

chris-chtrusch commented 1 year ago

https://www.tagesschau.de/resources/assets/fonts/TheSansC5s-4_SemiLightItalic.woff2 https://www.tagesschau.de/resources/assets/fonts/TheSansC5s-4_SemiLight.woff2 https://www.tagesschau.de/resources/assets/fonts/TheSansC5s-6_SemiBold.woff2

classilla commented 11 months ago

Noted and landing shortly