search

perusio / drupal-with-nginx

Running Drupal using nginx: an idiosyncratically crafted bleeding edge configuration.
855 stars 246 forks source link

Add woff2 extention to directly served static files #255

Closed iryston closed 8 years ago

iryston commented 8 years ago

This commit fixes fonts loading problems in certain configurations.

kkomelin commented 8 years ago

I met this issue with Glyphicon icon font, which is included in some versions of Bootstrap framework by default.

iryston commented 8 years ago

@kkomelin I had issues with Font Awesome and Google icons too.

kkomelin commented 8 years ago

So I vote for this PR

iryston commented 8 years ago

Thanks!

kkomelin commented 8 years ago

@perusio Could you please take a look at this PR when you have time. Thanks.

pprishchepa commented 8 years ago

@iryston What about woff2 in mime.types? It's only woff there.

kkomelin commented 8 years ago

Good point @PavelPrischepa

iryston commented 8 years ago

@PavelPrischepa thank you for PR! I want to pay attention that according to w3c recommendation WOFF 2.0 is an improvement on WOFF 1.0. The two formats have different Internet Media Types and may be used in parallel. So, WOFF2 mime type should be

application/font-woff2                 woff2;
perusio commented 8 years ago

@iryston I committed support for woff2 using woff2? instead of another option in the group. Thanks for your PR. https://github.com/perusio/drupal-with-nginx/commit/747fc306b86ff772829265ab475de7b9825aff7e

iryston commented 8 years ago

@perusio thanks for your commit.