Skip to content

Adapt global styles code to use get global styles functions instead of low-level APIs#36075

Merged
oandregal merged 2 commits intofix/settings-themes-no-theme-jsonfrom
update/use-get-global-styles-functions
Oct 29, 2021
Merged

Adapt global styles code to use get global styles functions instead of low-level APIs#36075
oandregal merged 2 commits intofix/settings-themes-no-theme-jsonfrom
update/use-get-global-styles-functions

Conversation

@oandregal
Copy link
Copy Markdown
Member

@oandregal oandregal commented Oct 29, 2021

This PR follows-up and it's built on top of #36054 (see discussion).

It adapts the code to prepare for a larger refactoring of the low-level APIs of global styles (remove the settings entirely).

@oandregal oandregal self-assigned this Oct 29, 2021
@oandregal oandregal added [Type] Code Quality Issues or PRs that relate to code quality Global Styles Anything related to the broader Global Styles efforts, including Styles Engine and theme.json labels Oct 29, 2021
@oandregal oandregal merged commit 0ec76f8 into fix/settings-themes-no-theme-json Oct 29, 2021
@oandregal oandregal deleted the update/use-get-global-styles-functions branch October 29, 2021 12:04
oandregal added a commit that referenced this pull request Nov 2, 2021
* Use gutenberg_get_global_* functions (#36075)

* Remove settings as an argument from `WP_Theme_JSON_Resolver` (#36081)
oandregal added a commit that referenced this pull request Nov 2, 2021
* Use gutenberg_get_global_* functions (#36075)

* Remove settings as an argument from `WP_Theme_JSON_Resolver` (#36081)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Global Styles Anything related to the broader Global Styles efforts, including Styles Engine and theme.json [Type] Code Quality Issues or PRs that relate to code quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant