Make WordPress Core

Ticket #24859: 24859.8.diff

File 24859.8.diff, 2.2 KB (added by kadamwhite, 10 years ago)

moved setting the initial spinner to false back to where gcorne had it

  • 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..d40587b 100644
     
    12901290        margin: 0;
    12911291}
    12921292
     1293.media-toolbar .spinner {
     1294        display: inline-block;
     1295        margin-top: 14px;
     1296}
     1297
    12931298.media-sidebar .settings-save-status {
    12941299        background: #f5f5f5;
    12951300        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..7893b5e 100644
     
    52485248                        });
    52495249
    52505250                        this.createToolbar();
     5251                        this.toggleSpinner( false );
    52515252                        this.updateContent();
    52525253                        this.createSidebar();
    52535254
    52545255                        this.collection.on( 'add remove reset', this.updateContent, this );
    52555256                },
     5257                toggleSpinner: function( state ) {
     5258                        if ( state ) {
     5259                                this.spinnerTimeout = _.delay(function( view ) {
     5260                                        view.toolbar.get( 'spinner' ).show();
     5261                                }, 600, this );
     5262                        } else {
     5263                                this.toolbar.get( 'spinner' ).hide();
     5264                                clearTimeout( this.spinnerTimeout );
     5265                        }
     5266                },
    52565267                /**
    52575268                 * @returns {wp.media.view.AttachmentsBrowser} Returns itself to allow chaining
    52585269                 */
     
    52895300                                }).render() );
    52905301                        }
    52915302
     5303                        this.toolbar.set( 'spinner', new media.view.Spinner({
     5304                                priority: -70
     5305                        }) );
     5306
    52925307                        if ( this.options.search ) {
    52935308                                this.toolbar.set( 'search', new media.view.Search({
    52945309                                        controller: this.controller,
     
    53135328                        }
    53145329
    53155330                        if ( ! this.collection.length ) {
    5316                                 this.collection.more().done( function() {
     5331                                this.toggleSpinner( true );
     5332                                this.collection.more().done(function() {
    53175333                                        if ( ! view.collection.length ) {
    53185334                                                view.createUploader();
    53195335                                        }
     5336                                        view.toggleSpinner( false );
    53205337                                });
    53215338                        }
    53225339                },
     
    63126329                        this.$( '.embed-media-settings' ).scrollTop( 0 );
    63136330                }
    63146331        });
     6332
     6333        /**
     6334         * wp.media.view.Spinner
     6335         *
     6336         *
     6337         * @constructor
     6338         * @augments wp.media.View
     6339         * @augments wp.Backbone.View
     6340         * @augments Backbone.View
     6341         */
     6342        media.view.Spinner = media.View.extend({
     6343                tagName:   'span',
     6344                className: 'spinner',
     6345
     6346                show: function() {
     6347                        this.$el.show();
     6348                        return this;
     6349                },
     6350
     6351                hide: function() {
     6352                        this.$el.hide();
     6353                        return this;
     6354                }
     6355        });
    63156356}(jQuery, _));