Changeset 41779

Timestamp:

10/06/2017 12:38:57 AM (9 years ago)

Author:

westonruter

Message:

Widgets: Apply the same container-constraining embed resizing logic from the Video widget to embeds in the Text widget.

See #40854, #39994.

File:

• trunk/src/wp-includes/widgets/class-wp-widget-text.php (modified) (1 diff)

trunk/src/wp-includes/widgets/class-wp-widget-text.php

@@ -291 +291 @@
                 }
 
+                $text = preg_replace_callback( '#<[^>]*>#', array( $this, 'inject_video_max_width_style' ), $text );
+
                 ?>
                         <div class="textwidget"><?php echo $text; ?></div>
                 <?php
                 echo $args['after_widget'];
+        }
+
+        /**
+         * Inject max-width and remove height for videos too constrained to fit inside sidebars on frontend.
+         *
+         * @since 4.9.0
+         *
+         * @see WP_Widget_Media_Video::inject_video_max_width_style()
+         * @param array $matches Pattern matches from preg_replace_callback.
+         * @return string HTML Output.
+         */
+        public function inject_video_max_width_style( $matches ) {
+                $html = $matches[0];
+                $html = preg_replace( '/\sheight="\d+"/', '', $html );
+                $html = preg_replace( '/\swidth="\d+"/', '', $html );
+                $html = preg_replace( '/(?<=width:)\s*\d+px(?=;?)/', '100%', $html );
+                return $html;
         }