Skip to content

WIP: Backport per-block global styles custom-CSS #3925

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.

Sign up for GitHub

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

Closed
aristath wants to merge 19 commits into from
Closed

WIP: Backport per-block global styles custom-CSS #3925

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
9d62e05
Add initial backports
glendaviesnz Jan 31, 2023
8693f2d
Add tests for global styles custom CSS
glendaviesnz Feb 1, 2023
cbcae3f
add debugging to unit test
glendaviesnz Feb 1, 2023
b8dd828
check for array or object
glendaviesnz Feb 1, 2023
6ce0630
Remove debugging
glendaviesnz Feb 1, 2023
9525323
Fix permissions issue on multisite test
glendaviesnz Feb 1, 2023
f090f7f
Put permissions assignment in the right place!
glendaviesnz Feb 1, 2023
25e3e9c
Set permissions in tests instead of setup to prevent changing permiss…
glendaviesnz Feb 1, 2023
8bf8b05
Fixes from code review
glendaviesnz Feb 1, 2023
5aff486
But make user definition that was removed by mistake
glendaviesnz Feb 1, 2023
3c7461b
Change ordering of $request[styles] assignment to $config[styles]
glendaviesnz Feb 1, 2023
2026c09
Add initial backports
glendaviesnz Jan 31, 2023
d6a2a7b
Refactor to match new individual custom CSS stylesheet
glendaviesnz Feb 1, 2023
b003ae7
add debugging to unit test
glendaviesnz Feb 1, 2023
369bda5
check for array or object
glendaviesnz Feb 1, 2023
dcfa519
Remove debugging
glendaviesnz Feb 1, 2023
80efb69
Add tests for custom CSS user capabilities
glendaviesnz Feb 1, 2023
6141cc3
Add initial backports from https://github.com/WordPress/gutenberg/pul…
glendaviesnz Jan 23, 2023
ad060f5
backport https://github.com/WordPress/gutenberg/pull/46571
aristath Jan 27, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fixes from code review
  • Loading branch information
@glendaviesnz @aristath
glendaviesnz authored and aristath committed Feb 3, 2023
commit 8bf8b05efcc0548c3c20015193e2e479d3aa6c29
8 changes: 5 additions & 3 deletions src/wp-includes/rest-api/endpoints/class-wp-rest-global-styles-controller.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -324,9 +324,11 @@ protected function prepare_item_for_database( $request ) {
}
}
$config['styles'] = $request['styles'];
$validate_custom_css = $this->validate_custom_css( $request['styles']['css'] );
if ( is_wp_error( $validate_custom_css ) ) {
return $validate_custom_css;
if ( isset( $config['styles']['css'] ) ) {
$css_validation_result = $this->validate_custom_css( $config['styles']['css'] );
if ( is_wp_error( $css_validation_result ) ) {
return $css_validation_result;
}
}
} elseif ( isset( $existing_config['styles'] ) ) {
$config['styles'] = $existing_config['styles'];
Expand Down