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

feat: add text option in google font params #66024

Open
wants to merge 1 commit into
base: canary
Choose a base branch
from

Conversation

Jean-Tinland
Copy link

Add text parameter to Google Font loader

What and why?

This PR enable text option support in next/font/google as it is described in Google developers documentation.

It allows more optimization by enabling partial font loading with only wanted characters.

I added some tests and an entry about this property in the corresponding documentation.

Please let me know if this PR is missing anything, I'll make the necessary adjustments.

@Jean-Tinland Jean-Tinland requested review from manovotny and delbaoliveira and removed request for a team May 21, 2024 08:49
@ijjk ijjk added Documentation Related to Next.js' official documentation. Font (next/font) Related to Next.js Font Optimization. type: next labels May 21, 2024
@ijjk
Copy link
Member

ijjk commented May 21, 2024

Allow CI Workflow Run

  • approve CI run for commit: 11b07f6

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Related to Next.js' official documentation. Font (next/font) Related to Next.js Font Optimization. type: next
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants