Skip to content

Font Library: add font family and font face preview keys to schema#56793

Merged
matiasbenedetto merged 4 commits intotrunkfrom
add/font-preview-to-schema
Dec 5, 2023
Merged

Font Library: add font family and font face preview keys to schema#56793
matiasbenedetto merged 4 commits intotrunkfrom
add/font-preview-to-schema

Conversation

@matiasbenedetto
Copy link
Copy Markdown
Contributor

@matiasbenedetto matiasbenedetto commented Dec 5, 2023

What?

Font Library: add font family and font face preview keys to schema.
The preview property is optional and it's used to store a reference to a preview of the font asset.

Example:
In the Font Library default font collection is used to store an image preview in SVG format.

Why?

To avoid using a non-documented key preview.
To avoid wiping the preview data during sanitization.

How?

Add to theme.json schema to document this key.
Add to PHP sanitization schema to avoid the preview data being removed by the sanitization process.

Testing Instructions

  • Install a font from the Google Fonts collection.
  • Check that the font preview data is not removed when saved to the database.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 5, 2023

This pull request has changed or added PHP files. Please confirm whether these changes need to be synced to WordPress Core, and therefore featured in the next release of WordPress.

If so, it is recommended to create a new Trac ticket and submit a pull request to the WordPress Core Github repository soon after this pull request is merged.

If you're unsure, you can always ask for help in the #core-editor channel in WordPress Slack.

Thank you! ❤️

View changed files
❔ lib/class-wp-theme-json-gutenberg.php

@matiasbenedetto matiasbenedetto added the [Type] Bug An existing feature does not function as intended label Dec 5, 2023
@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 5, 2023

Flaky tests detected in 2616cfc.
Some tests passed with failed attempts. The failures may not be related to this commit but are still reported for visibility. See the documentation for more information.

🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/7106910502
📝 Reported issues:

Copy link
Copy Markdown
Contributor

@jffng jffng left a comment

Choose a reason for hiding this comment

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

The docs need to be rebuilt (npm run docs:build), other than that and one small comment, LGTM.

matiasbenedetto and others added 2 commits December 5, 2023 15:37
Co-authored-by: Jeff Ong <jonger4@gmail.com>
@matiasbenedetto
Copy link
Copy Markdown
Contributor Author

The docs need to be rebuilt (npm run docs:build), other than that and one small comment, LGTM.

Thanks for the review. Docs updated :)

@matiasbenedetto matiasbenedetto enabled auto-merge (squash) December 5, 2023 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Bug An existing feature does not function as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants