feat(fonts): new cssVariable property

[florian-lefebvre]

Changes

• Addresses feedback from the API bash
• Removes the as prop. Instead, we generate a new nameWithHash (hash computed from the family config object) property to avoid conflicts if the same font name is specified more than once
• Adds a new required cssVariable property. That gives control to the user and avoids having to guess. There's minimal validation, that should handle most mistakes
• It simplifies a lot of things by removing some checks and errors that are now unecessary

Testing

Manual and automated

Docs

RFC updated

[changeset-bot]

⚠️ No Changeset found

Latest commit: 8bcc1d0

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[delucis]

Cool seeing the deleted code here! 🙌

[delucis]

Looking good I think!

[ematipico]

We need to handle errors coming from the resolution of font providers