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

Font Library: WordPress 6.5 Required Tasks #55277

Closed
25 tasks done
matiasbenedetto opened this issue Oct 11, 2023 · 17 comments
Closed
25 tasks done

Font Library: WordPress 6.5 Required Tasks #55277

matiasbenedetto opened this issue Oct 11, 2023 · 17 comments
Labels
[Feature] Font Library [Feature] Typography Font and typography-related issues and PRs [Type] Iteration Scoped iteration of an effort from a tracking issue or overview issue ideally for a major release. [Type] Tracking Issue Tactical breakdown of efforts across the codebase and/or tied to Overview issues.

Comments

@matiasbenedetto
Copy link
Contributor

matiasbenedetto commented Oct 11, 2023

What?

A list of tasks required to merge Font Library in WordPress core 6.5.

To do:

In progress:

Needs decision:

Done:

Nice to have:

Trac ticket to merge in core:

https://core.trac.wordpress.org/ticket/59166

@matiasbenedetto matiasbenedetto added [Type] Tracking Issue Tactical breakdown of efforts across the codebase and/or tied to Overview issues. [Feature] Typography Font and typography-related issues and PRs labels Oct 11, 2023
@matiasbenedetto matiasbenedetto changed the title Font Library: 6.5 required tasks Font Library: WordPress core 6.5 release required tasks Oct 11, 2023
@annezazu annezazu moved this to Punted to 6.5 in WordPress 6.4 Editor Tasks Oct 11, 2023
@priethor priethor added the [Type] Iteration Scoped iteration of an effort from a tracking issue or overview issue ideally for a major release. label Dec 4, 2023
@SaxonF SaxonF moved this to In Progress in Design priorities Dec 7, 2023
@alexandrebuffet
Copy link
Contributor

Hello there. I opened ticket 59911 on the track concerning inconsistency brought by PR #54615 between the declaration of font family presets in theme.json and the generated @font-face. Can we continue the discussion in #57207?

@annezazu
Copy link
Contributor

annezazu commented Jan 2, 2024

👋🏼 Hey hey! Are you all able to share biweekly updates on how this work is progressing? It would help me loads with sharing updates to the broader community and coordinating testing as needed.

@matiasbenedetto
Copy link
Contributor Author

👋🏼 Hey hey! Are you all able to share biweekly updates on how this work is progressing? It would help me loads with sharing updates to the broader community and coordinating testing as needed.

Yes, here's an update from the last few weeks:

Changes merged:

#57282 to consolidate the existing endpoints the Font Library added and fix potential data sanitization problems. This change is a small step forward in implementing the API refactor.

#57364 to eliminate code no longer required.

#57368 to improve copy text.

Most relevant changes in-process:

@annezazu annezazu moved this to 📥 Todo in WordPress 6.5 Editor Tasks Jan 5, 2024
@mikachan mikachan changed the title Font Library: WordPress core 6.5 release required tasks Font Library: WordPress 6.5 Required Tasks Jan 5, 2024
@mikachan mikachan moved this from 📥 Todo to 🏗️ In Progress in WordPress 6.5 Editor Tasks Jan 12, 2024
@annezazu
Copy link
Contributor

@matiasbenedetto are you able to provide weekly updates at this stage? We're less than a month away from beta 1 and it's going to be critical that we're aware of how this feature is progressing 🌟

@matiasbenedetto
Copy link
Contributor Author

@annezazu yes, this is the report about the last two weeks:

Changes discussed or merged:

We continue working heavily of the API rest refactor and the required frontend changes around that.
#57688

We discussed the requirement around a method for the extenders to turn off the library. We landed a solution consisting of adding a PHP filter: #55275.

We discussed the need for a custom permission for managing fonts, and it seems like it has been discarded at least for now:
#55280

We discussed and landed a solution around configuring the storage location for font assets:
#57697

We added the ability to unregister font collections:
#54701

We updated the Google fonts collection generation and replaced the older TTF files by the lighter WOFF2 files.
https://github.com/WordPress/google-fonts-to-wordpress-collection

Most relevant for the next days:

Work on the feature branch around the API refactor that will include the frontend changes too and try to merge it to trunk.
#57688

@mikachan
Copy link
Member

Quick update on the Font Library: the API refactor PR has been merged, ready for release with Gutenberg 17.6. We'll start on the backports for 6.5 soon.

@colorful-tones
Copy link
Member

Consider this as a 'Nice to Have' Font Library: Global Styles > Typography sidebar – group related font sources #58403

@getdave
Copy link
Contributor

getdave commented Jan 31, 2024

As Editor Tech Leads, @youknowriad and I have been looking at how we can best support contributors with landing this (blessed) feature for the WordPress 6.5 release.

I've spent some time looking at the code and how we could possibly seek to make the synchronisation to WP Core easier for folks to review. It's our feeling that it might be easier/more manageable to split the feature into its high level components and merge incrementally.

I've taken the liberty of pulling together the following. It is a suggestion only and we will defer to contributors on how best to proceed so feel free to pull it apart and propose alternatives.

Know that we are here to support you, so please reach out to and we're happy to jump in and assist as required.

Thanks for all the work on this feature 🙇

Proposal

The items are listed in order of proposed merge into WP Core.

Post Types registration and Low Level APIs

All Font Library post type registrations.

These are required to allow the public facing elements (REST API and Post Types) to function.

REST API endpoints

The classes relating to REST need to be available before the Post Type registration is merge otherwise the rest_controller_class config will fail (example).

Public API methods

All remaining methods/functions from font-library.php.

@mikachan
Copy link
Member

Another update:

  • We have made further smaller changes to the Font Library (both PHP and JS) over the past week, ready to be released with Gutenberg 17.6.
  • We plan to open 2 PRs for Core:
    1. General PHP: Custom post type registration, Font Library, Font Collection classes and functions
    2. REST API controllers
  • The preparation for those is happening now, and we hope to have a further update by the end of this week (Feb 2).

If you'd like to follow along, all the work is labeled with [Feature] Font Library.

@getdave
Copy link
Contributor

getdave commented Feb 2, 2024

Update

In order to raise the x2 Core PRs we need the following:

It would also be nice to have the following (but these are not blockers to raising the Core PRs):

@creativecoder
Copy link
Contributor

creativecoder commented Feb 5, 2024

Two more to add to the above to prep for the Core PRs

@youknowriad
Copy link
Contributor

youknowriad commented Feb 5, 2024

Initial Font Library Core backport PR is open here WordPress/wordpress-develop#6027

@getdave
Copy link
Contributor

getdave commented Feb 5, 2024

Some additional feedback on the code is being addressed in Gutenberg and will need to be brought into the Core PR(s).

@getdave
Copy link
Contributor

getdave commented Feb 6, 2024

Core PRs for the initial merge of this feature are:

@getdave
Copy link
Contributor

getdave commented Feb 7, 2024

The initial merge of this feature into WordPress Core for the 6.5 release is complete. Thank you to everyone who contributed.

@creativecoder @matiasbenedetto if you are able to ✅ off any PRs from the PHP Sync Issue and mark with Backported to WP Core label that would be appreciated.

@fabiankaegy
Copy link
Member

Now that we are past the Beta 1 cut off point I'm removing this tracking issue from the 6.5 Project board. From now on the board should only contain individual bugs that we want to still fix before 6.5 gets released.

@mikachan
Copy link
Member

I'm going to close this issue, as the tasks listed above have now all been completed and the feature is due to be released with WP 6.5. Further issues can be tracked using the [Feature] Font Library label.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Font Library [Feature] Typography Font and typography-related issues and PRs [Type] Iteration Scoped iteration of an effort from a tracking issue or overview issue ideally for a major release. [Type] Tracking Issue Tactical breakdown of efforts across the codebase and/or tied to Overview issues.
Projects
None yet
Development

No branches or pull requests

10 participants