move to local fonts b/c the alternative is dumb #137
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fixes #61
We were using next/fonts and users were sometimes unable to download the fonts causing fallbacks to render.
Here I've moved to only local fonts for the project.
This involved:
<PageHeading>
component that applies a local heading fontextendTheme
function. If that wasn't the case we could have defined the heading and body fonts there instead of injecting them in a custom<PageHeader>
component, or importing them within the app.