Changeset 41117 for branches/4.8/src/wp-includes/default-widgets.php

Timestamp:

07/21/2017 09:16:58 PM (8 years ago)

Author:

westonruter

Message:

Widgets: Improve theme styling compatibility for Custom HTML widget by adding content container with textwidget class and widget_text class on widget wrapper element.

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.

Merges [41115] and partially [41116] onto 4.8 branch.
Amends [40893].
See #40907.
Fixes #41392 for 4.8.1.

Location:

branches/4.8

Files:

• . (modified) (1 prop)
• src/wp-includes/default-widgets.php (modified) (1 diff)

branches/4.8/src/wp-includes/default-widgets.php

@@ -134 +134 @@
         $content = apply_filters( 'widget_custom_html_content', $content, $instance, $this );
 
+        // Inject the Text widget's container class name alongside this widget's class name for theme styling compatibility.
+        $args['before_widget'] = preg_replace( '/(?<=\sclass=["\'])/', 'widget_text ', $args['before_widget'] );
+
         echo $args['before_widget'];
         if ( ! empty( $title ) ) {
             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'];
     }