Skip to content

Backport: add theme json class changes to edit style variations #3941

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
tellthemachines wants to merge 12 commits into from
Closed

Backport: add theme json class changes to edit style variations #3941

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
53ce333
Add theme json class changes to edit style variations
tellthemachines Jan 30, 2023
c99dace
Add test for style variations
oandregal Jan 30, 2023
2c58b47
fixup
oandregal Jan 30, 2023
923feca
Merge pull request #1 from oandregal/backport/edit-style-variations
tellthemachines Jan 31, 2023
fe1353c
Fix test output styles.
tellthemachines Jan 31, 2023
80ad395
Fix lint errors.
tellthemachines Jan 31, 2023
1f232da
Fix spacing in test file.
tellthemachines Jan 31, 2023
159a3e3
Remove trailing comma to resolve parsing error in < PHP 7.3.
hellofromtonya Jan 31, 2023
fc4a2a2
Tests: split and use data provider
hellofromtonya Jan 31, 2023
86ac232
Guard for non-array variations
hellofromtonya Jan 31, 2023
994c202
Micro-optimizations in ::get_styles_for_block()
hellofromtonya Jan 31, 2023
0768aa8
Formatting for multiline comments
hellofromtonya Jan 31, 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
Fix lint errors.
  • Loading branch information
@tellthemachines
tellthemachines committed Jan 31, 2023
commit 80ad395ba44dd1e090ae6c74f17129fd3e57bc19
1 change: 0 additions & 1 deletion src/wp-includes/class-wp-theme-json.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2289,7 +2289,6 @@ function( $pseudo_selector ) use ( $selector ) {
$block_rules .= static::to_ruleset( $style_variation_selector, $individual_style_variation_declarations );
}


return $block_rules;
}

Expand Down
2 changes: 1 addition & 1 deletion tests/phpunit/tests/theme/wpThemeJson.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3610,7 +3610,7 @@ public function test_get_styles_for_block_with_style_variations() {
'color' => array(
'background' => 'hotpink',
),
'unregisteredProperty' => 'value'
'unregisteredProperty' => 'value',
),
),
),
Expand Down