WordPress.org

Plugin Directory

Changeset 623869


Ignore:
Timestamp:
11/12/12 03:56:20 (17 months ago)
Author:
mdawaffe
Message:

Admin: Ensure "More Info" box is large enough to handle translated text.

By juggling height and min-height, we can make the box have arbitrary height and still slide up and down.
Set the height to "auto" when displaying

Location:
jetpack/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • jetpack/trunk/_inc/jetpack.js

    r622362 r623869  
    143143                    } else { 
    144144                        jQuery( 'div.more-info div.jp-content' ).hide(); 
    145                         jQuery( 'div.more-info' ).slideUp( 200, function() { 
    146                             jQuery(this).detach().insertAfter( el ); 
     145                        jQuery( 'div.more-info' ).css( { height: '230px', minHeight: 0 } ).slideUp( 200, function() { 
     146                            var $this = jQuery(this); 
     147                            $this.detach().insertAfter( el ); 
    147148                            jQuery( 'div.more-info div.jp-content' ).hide(); 
    148149                            jetpack.learn_more_content( jQuery(card).attr( 'id' ) ); 
    149                             jQuery( 'div.more-info' ).slideDown( 300 ); 
    150                             jQuery( window ).scrollTo( ( jQuery( 'div.more-info' ).prev().offset().top ) - 70, 600, function() { if ( typeof callback == 'function' ) callback.call( this ); } ); 
     150                            $this.css( { height: '230px', minHeight: 0 } ).slideDown( 300, function() { 
     151                                $this.css( { height: 'auto', minHeight: '230px' } ); 
     152                            } ); 
     153                            jQuery( window ).scrollTo( ( $this.prev().offset().top ) - 70, 600, function() { if ( typeof callback == 'function' ) callback.call( this ); } ); 
    151154                        } ); 
    152155                    } 
     
    158161 
    159162                    // Show the box 
     163                    jQuery( 'div.more-info' ).css( { height: '230px', minHeight: 0 } ); 
    160164                    jQuery( 'div.more-info', 'div.module-container' ).hide().slideDown( 400, function() { 
     165                        jQuery( 'div.more-info' ).css( { height: 'auto', minHeight: '230px' } ); 
    161166                        // Load the content and scroll to it 
    162167                        jetpack.learn_more_content( jQuery(card).attr( 'id' ) ); 
     
    214219        jQuery( 'div.more-info div.jp-content' ).hide(); 
    215220 
    216         jQuery( 'div.more-info' ).slideUp( 200, function() { 
     221        jQuery( 'div.more-info' ).css( { height: '230px', minHeight: 0 } ).slideUp( 200, function() { 
    217222            jQuery( this ).remove(); 
    218223                jQuery( 'a.jetpack-deactivate-button' ).hide(); 
  • jetpack/trunk/jetpack.php

    r623227 r623869  
    15401540 
    15411541    function admin_scripts() { 
    1542         wp_enqueue_script( 'jetpack-js', plugins_url( basename( dirname( __FILE__ ) ) ) . '/_inc/jetpack.js', array( 'jquery' ), JETPACK__VERSION . '-20120805' ); 
     1542        wp_enqueue_script( 'jetpack-js', plugins_url( basename( dirname( __FILE__ ) ) ) . '/_inc/jetpack.js', array( 'jquery' ), JETPACK__VERSION . '-20121111' ); 
    15431543        wp_localize_script( 'jetpack-js', 'jetpackL10n', array( 
    15441544                'ays_disconnect' => "This will deactivate all Jetpack modules.\nAre you sure you want to disconnect?", 
Note: See TracChangeset for help on using the changeset viewer.