Changeset 41115

Timestamp:

07/21/2017 06:12:43 PM (7 years ago)

Author:

westonruter

Message:

Widgets: Improve theme styling compatibility for Custom HTML widget by adding container with textwidget class.

The same styling from the Text widget should apply to the Custom HTML widget since users are expected to copy HTML from the (legacy) Text widget into the latter.

Amends [40893].
See #40907.
Fixes #41392.

Location:

trunk

Files:

• src/wp-includes/widgets/class-wp-widget-custom-html.php (modified) (1 diff)
• tests/phpunit/tests/widgets/custom-html-widget.php (modified) (1 diff)

trunk/src/wp-includes/widgets/class-wp-widget-custom-html.php

@@ -80 +80 @@
             echo $args['before_title'] . $title . $args['after_title'];
         }
+        echo '<div class="textwidget custom-html-widget">'; // The textwidget class is for theme styling compatibility.
         echo $content;
+        echo '</div>';
         echo $args['after_widget'];
     }

trunk/tests/phpunit/tests/widgets/custom-html-widget.php

@@ -71 +71 @@
         $this->assertNotEmpty( $this->widget_text_args );
         $this->assertContains( '[filter:widget_text][filter:widget_custom_html_content]', $output );
+        $this->assertContains( '<div class="textwidget custom-html-widget">', $output );
         $this->assertNotContains( '<p>', $output );
         $this->assertNotContains( '<br>', $output );