Make WordPress Core

Ticket #24859: 24859.9.diff

File 24859.9.diff, 2.1 KB (added by kadamwhite, 11 years ago)

make gcorne's final css adjustment to avoid having to manually suppress the spinner

  • src/wp-includes/css/media-views.css

    diff --git src/wp-includes/css/media-views.css src/wp-includes/css/media-views.css
    index 9a4478a..b653be5 100644
     
    12901290        margin: 0;
    12911291}
    12921292
     1293.media-toolbar .spinner {
     1294        margin-top: 14px;
     1295}
     1296
    12931297.media-sidebar .settings-save-status {
    12941298        background: #f5f5f5;
    12951299        float: right;
  • src/wp-includes/js/media-views.js

    diff --git src/wp-includes/js/media-views.js src/wp-includes/js/media-views.js
    index c3f15f9..2e0922b 100644
     
    52535253
    52545254                        this.collection.on( 'add remove reset', this.updateContent, this );
    52555255                },
     5256                toggleSpinner: function( state ) {
     5257                        if ( state ) {
     5258                                this.spinnerTimeout = _.delay(function( view ) {
     5259                                        view.toolbar.get( 'spinner' ).show();
     5260                                }, 600, this );
     5261                        } else {
     5262                                this.toolbar.get( 'spinner' ).hide();
     5263                                clearTimeout( this.spinnerTimeout );
     5264                        }
     5265                },
    52565266                /**
    52575267                 * @returns {wp.media.view.AttachmentsBrowser} Returns itself to allow chaining
    52585268                 */
     
    52895299                                }).render() );
    52905300                        }
    52915301
     5302                        this.toolbar.set( 'spinner', new media.view.Spinner({
     5303                                priority: -70
     5304                        }) );
     5305
    52925306                        if ( this.options.search ) {
    52935307                                this.toolbar.set( 'search', new media.view.Search({
    52945308                                        controller: this.controller,
     
    53135327                        }
    53145328
    53155329                        if ( ! this.collection.length ) {
    5316                                 this.collection.more().done( function() {
     5330                                this.toggleSpinner( true );
     5331                                this.collection.more().done(function() {
    53175332                                        if ( ! view.collection.length ) {
    53185333                                                view.createUploader();
    53195334                                        }
     5335                                        view.toggleSpinner( false );
    53205336                                });
    53215337                        }
    53225338                },
     
    63126328                        this.$( '.embed-media-settings' ).scrollTop( 0 );
    63136329                }
    63146330        });
     6331
     6332        /**
     6333         * wp.media.view.Spinner
     6334         *
     6335         * @constructor
     6336         * @augments wp.media.View
     6337         * @augments wp.Backbone.View
     6338         * @augments Backbone.View
     6339         */
     6340        media.view.Spinner = media.View.extend({
     6341                tagName:   'span',
     6342                className: 'spinner',
     6343
     6344                show: function() {
     6345                        this.$el.show();
     6346                        return this;
     6347                },
     6348
     6349                hide: function() {
     6350                        this.$el.hide();
     6351                        return this;
     6352                }
     6353        });
    63156354}(jQuery, _));