Skip to content

Docs: Updated note about using next/head in basic-features/font-optimization#39863

Merged
ijjk merged 2 commits intovercel:canaryfrom
hanneslund:font-optimization-docs-note
Aug 23, 2022
Merged

Docs: Updated note about using next/head in basic-features/font-optimization#39863
ijjk merged 2 commits intovercel:canaryfrom
hanneslund:font-optimization-docs-note

Conversation

@hanneslund
Copy link
Copy Markdown
Contributor

Documentation / Examples

  • Make sure the linting passes by running pnpm lint
  • The examples guidelines are followed from our contributing doc

Copy link
Copy Markdown
Member

@ijjk ijjk left a comment

Choose a reason for hiding this comment

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

Thanks!

@ijjk ijjk merged commit ca9fef6 into vercel:canary Aug 23, 2022
```

Note that we don't recommend adding fonts with `next/head`, as this only applies the font to the particular page and won't work with a streaming architecture.
Although it's possible to add fonts in a page with [`next/head`](/docs/api-reference/next/head) it comes with some drawbacks:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Slight update: #39950

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants