Financial-Times / n-service-worker

❌ DECOMMISSIONED Global service worker component for next.ft.com
5 stars 2 forks source link

removing question marks in a bid to prevent downloading each font twice #130

Closed bjfletcher closed 6 years ago

bjfletcher commented 7 years ago

... on first load (by o-fonts CSS in our critical CSS, and again by Service Worker)

wheresrhys commented 6 years ago

Ah good spot. but it may actually be the case that wherever uses the url without ? should have them added. Might be there to address some odd bug.

@quarterto do you know anything about these ??

wheresrhys commented 6 years ago

Might be a cack-handed nod to this http://blog.fontspring.com/2011/02/the-new-bulletproof-font-face-syntax/

apaleslimghost commented 6 years ago

this was for appcache and ?#iefix in o-fonts, which isn't there any more as of Financial-Times/o-fonts#76. go ahead and remove it