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

[Fonts API] Remove all files once Font Library is merged #51820

Closed
hellofromtonya opened this issue Jun 22, 2023 · 0 comments · Fixed by #57972
Closed

[Fonts API] Remove all files once Font Library is merged #51820

hellofromtonya opened this issue Jun 22, 2023 · 0 comments · Fixed by #57972
Assignees
Labels
[Feature] Typography Font and typography-related issues and PRs [Type] Task Issues or PRs that have been broken down into an individual action to take

Comments

@hellofromtonya
Copy link
Contributor

Related to #51769 and #51819

Part of #41479.

Dependent upon #51770, #51819, and #50927 being merged into trunk.

What problem does this address?

The Fonts API will be dead code once the Font Library is merged into trunk, as it will not load into memory.

What is your proposed solution?

Once Font Library, Font Face, and #51819 are all merged into trunk, then:

  1. Delete the entire /lib/experimental/fonts-api/ folder.
  2. Delete the require lines for each of those files in lib/load.php.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Typography Font and typography-related issues and PRs [Type] Task Issues or PRs that have been broken down into an individual action to take
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants