Changeset 57260

Timestamp:

01/10/2024 12:25:19 AM (18 months ago)

Author:

isabel_brison

Message:

Editor: Allow default duotone styles if not explicitly disabled in theme.json.

Removes setting that disabled default duotone palette from being output in themes without theme.json.

Props andrewserong.
Fixes #60136.

Location:

trunk

Files:

• src/wp-includes/class-wp-theme-json-resolver.php (modified) (1 diff)
• tests/phpunit/tests/theme/wpThemeJsonResolver.php (modified) (1 diff)

trunk/src/wp-includes/class-wp-theme-json-resolver.php

@@ -312 +312 @@
             }
             $theme_support_data['settings']['color']['defaultGradients'] = $default_gradients;
-
-            // Classic themes without a theme.json don't support global duotone.
-            $theme_support_data['settings']['color']['defaultDuotone'] = false;
 
             // Allow themes to enable link color setting via theme_support.

trunk/tests/phpunit/tests/theme/wpThemeJsonResolver.php

@@ -471 +471 @@
         $this->assertSame( $color_palette, $settings['color']['palette']['theme'] );
         $this->assertTrue( $settings['border']['color'], 'Support for "appearance-tools" was not added.' );
+    }
+
+    /**
+     * Tests that classic themes still get core default settings such as color palette and duotone.
+     *
+     * @ticket 60136
+     */
+    public function test_core_default_settings_are_loaded_for_themes_without_theme_json() {
+        switch_theme( 'default' );
+
+        $settings = WP_Theme_JSON_Resolver::get_merged_data( 'theme' )->get_settings();
+
+        $this->assertFalse( wp_theme_has_theme_json() );
+        $this->assertTrue( $settings['color']['defaultPalette'] );
+        $this->assertTrue( $settings['color']['defaultDuotone'] );
+        $this->assertTrue( $settings['color']['defaultGradients'] );
     }