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

Typography block supports: sync with 6.1 backport #43928

Merged
merged 1 commit into from
Dec 19, 2022

Conversation

ramonjd
Copy link
Member

@ramonjd ramonjd commented Sep 7, 2022

What?

Removing deprecated function and updating comments to sync with 6.1 Core backport PR WordPress/wordpress-develop#3203

wp_typography_get_css_variable_inline_style was only deprecated in 6.1, and the Gutenberg plugin supports a minimum of 6.0.

This PR now only updates the comments instead of removing the method.

Why?

To make it clear in the comments that we can remove gutenberg_typography_get_css_variable_inline_style once the min supported version of WP in the plugin is 6.1. 😄

@ramonjd ramonjd added Backport from WordPress Core Pull request that needs to be backported to a Gutenberg release from WordPress Core [Feature] Typography Font and typography-related issues and PRs labels Sep 7, 2022
@ramonjd ramonjd self-assigned this Sep 7, 2022
@ramonjd ramonjd force-pushed the update/typography-block-support-sync-with-6-1 branch from 67ba25f to 46bf0da Compare September 7, 2022 05:26
@ramonjd ramonjd force-pushed the update/typography-block-support-sync-with-6-1 branch from 46bf0da to 5e392ae Compare September 21, 2022 05:02
@ramonjd ramonjd force-pushed the update/typography-block-support-sync-with-6-1 branch from 5e392ae to a7c1654 Compare December 19, 2022 06:25
The reason for revert is that `wp_typography_get_css_variable_inline_style` was only deprecated in 6.1, and the Gutenberg plugin supports a minimum of 6.0.
@ramonjd ramonjd force-pushed the update/typography-block-support-sync-with-6-1 branch from a7c1654 to 5d8fc2c Compare December 19, 2022 06:30
@ramonjd ramonjd enabled auto-merge (squash) December 19, 2022 06:31
Copy link
Contributor

@tellthemachines tellthemachines left a comment

Choose a reason for hiding this comment

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

Changes look good! Might need to prettify the spacing on the gutenberg_typography_get_preset_inline_style_value params for linter to be happy.

@ramonjd ramonjd merged commit f6cccc2 into trunk Dec 19, 2022
@ramonjd ramonjd deleted the update/typography-block-support-sync-with-6-1 branch December 19, 2022 07:07
@github-actions github-actions bot added this to the Gutenberg 14.9 milestone Dec 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backport from WordPress Core Pull request that needs to be backported to a Gutenberg release from WordPress Core [Feature] Typography Font and typography-related issues and PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants