Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

theme is missing font fa-brands-400.woff2 #44

Closed
wants to merge 2 commits into from

Conversation

ndy40
Copy link

@ndy40 ndy40 commented Jan 30, 2021

The theme is missing the font static/webfonts/fa-brands-400.woff2.

This sometimes causes the page not to display correctly on some phones with certain browser.

Had a friend view a Hugo site I was trying out and this got flagged. Not a problem in normal browsers.

@ndy40 ndy40 changed the title theme is missing font theme is missing font fa-brands-400.woff2 Jan 30, 2021
@ndy40 ndy40 changed the title theme is missing font fa-brands-400.woff2 theme is missing font fa-brands-400.woff2 Jan 30, 2021
@dataCobra
Copy link
Owner

Hey @ndy40,

thanks for pointing me at this problem. I did remove all the font files which are not "woff" on purpose,
because I wanted to reduce the size of the theme. Since the new version of FontAwesome came out, I will
update the theme now and re-add all the missing font-files.

Thanks for your effort to even create a PR, but because of the upcoming update of FontAwesome anyway, your PR is not needed.

Regards, Ben

@dataCobra dataCobra closed this Jan 30, 2021
@dataCobra
Copy link
Owner

When I'm finished with the version update of FontAwesome I'll create one more comment here. 👍

@dataCobra
Copy link
Owner

The release is now available and you either download the stable release or clone the repository. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants