Changeset 4385

Timestamp:

11/21/2016 02:27:26 AM (7 years ago)

Author:

dd32

Message:

Plugin Directory: Expand the initial description to show the full contents of any embed/videos present in the first paragraph/div.
This fixes cases where descriptions start with a video, and caused the controls to be hidden below the fold.

Fixes #1789.

File:

• sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-plugins/js/section-accordion.js (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-plugins/js/section-accordion.js

@@ -17 +17 @@
         initial_size: function( selector ) {
             $( selector ).each( function( i, el) {
-                if ( $(el).height() / el.scrollHeight > 0.8 || el.id == 'screenshots' ) {
+                var $el = $(el);
+                if ( $el.height() / el.scrollHeight > 0.8 || el.id == 'screenshots' ) {
                     // Force the section to expand, and hide its button
-                    $(el).toggleClass( 'toggled' ).addClass('short-content').attr( 'aria-expanded', true );
+                    $el.toggleClass( 'toggled' ).addClass('short-content').attr( 'aria-expanded', true );
                     $( '.section-toggle[aria-controls="' + el.id + '"]' ).hide();
                 } else {
+                    // If the description starts with an embed/video, set the min-height to include it.
+                    if ( 'description' == el.id && $el.children().next('p,div').first().find('video,iframe') ) {
+                        var height = $el.children().next('p,div').first().outerHeight(true) /* embed */ + $el.children().first().outerHeight(true) /* h2 */;
+
+                        if ( height > parseInt($el.css( 'max-height' )) ) {
+                            $el.css( 'min-height', height + "px" );
+                        }
+                    }
+
                     // Contract the section and make sure its button is visible
-                    $(el).removeClass( 'short-content' ).attr( 'aria-expanded', false );
+                    $el.removeClass( 'short-content' ).attr( 'aria-expanded', false );
                     $( '.section-toggle[aria-controls="' + el.id + '"]' ).show();
                 }