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

chore(docs): replace typefaces with fontsource #27313

Merged
merged 7 commits into from
Nov 17, 2020

Conversation

ayuhito
Copy link
Contributor

@ayuhito ayuhito commented Oct 6, 2020

Description

This PR changes the documentation for Gatsby's local hosting font solutions which was previously dependent on Typefaces to Fontsource.

Kyle and I were discussing this privately and he stated that he is deprecating the Typefaces project and no longer maintaining it in favour of Fontsource. The latest commits on his repo should reflect that and we're working on making a couple last changes.

Documentation

The two docs that were changed:

@gatsbyjs/documentation

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Oct 6, 2020
@KyleAMathews
Copy link
Contributor

Just finished updating the typeface packages w/ the deprecation notice https://www.npmjs.com/package/typeface-vesper-libre

KyleAMathews
KyleAMathews previously approved these changes Oct 6, 2020
@LekoArts LekoArts added type: documentation An issue or pull request for improving or updating Gatsby's documentation and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Oct 7, 2020
@ascorbic ascorbic added the status: needs docs review Pull request related to documentation waiting for review label Oct 8, 2020
Copy link
Contributor

@meganesu meganesu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi there! Thanks for your patience while I reviewed this. I made a few suggestions for improvements. Once those are resolved we can go ahead and get this merged!

Thanks for your contribution! 💜

docs/docs/recipes/styling-css.md Outdated Show resolved Hide resolved
docs/docs/recipes/styling-css.md Outdated Show resolved Hide resolved
docs/docs/recipes/styling-css.md Outdated Show resolved Hide resolved
docs/docs/recipes/styling-css.md Outdated Show resolved Hide resolved
docs/docs/recipes/styling-css.md Outdated Show resolved Hide resolved
docs/docs/recipes/styling-css.md Outdated Show resolved Hide resolved
docs/docs/recipes/styling-css.md Show resolved Hide resolved
docs/docs/recipes/styling-css.md Outdated Show resolved Hide resolved
docs/docs/using-web-fonts.md Show resolved Hide resolved
Co-authored-by: Megan Sullivan <meganesulli@gmail.com>
@ayuhito ayuhito requested a review from meganesu November 15, 2020 19:24
@ayuhito
Copy link
Contributor Author

ayuhito commented Nov 15, 2020

Should be almost good to go!

Copy link
Contributor

@meganesu meganesu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for making those changes. Looks good to me! ✨

Should be almost good to go!

Were there any other changes you wanted to make, or can I merge this?

@ayuhito
Copy link
Contributor Author

ayuhito commented Nov 17, 2020

@meganesu, nothing else comes to mind. LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: needs docs review Pull request related to documentation waiting for review type: documentation An issue or pull request for improving or updating Gatsby's documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants