Make WordPress Core

Ticket #24859: 24859.7.diff

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

Made it work in my environments, removed code that didn't seem to be needed, and style tweaks

  • 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..5306468 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,
     
    53085322                updateContent: function() {
    53095323                        var view = this;
    53105324
     5325                        view.toggleSpinner( false );
     5326
    53115327                        if( ! this.attachments ) {
    53125328                                this.createAttachments();
    53135329                        }
    53145330
    53155331                        if ( ! this.collection.length ) {
    5316                                 this.collection.more().done( function() {
     5332                                this.toggleSpinner( true );
     5333                                this.collection.more().done(function() {
    53175334                                        if ( ! view.collection.length ) {
    53185335                                                view.createUploader();
    53195336                                        }
     5337                                        view.toggleSpinner( false );
    53205338                                });
    53215339                        }
    53225340                },
     
    63126330                        this.$( '.embed-media-settings' ).scrollTop( 0 );
    63136331                }
    63146332        });
     6333
     6334        /**
     6335         * wp.media.view.Spinner
     6336         *
     6337         *
     6338         * @constructor
     6339         * @augments wp.media.View
     6340         * @augments wp.Backbone.View
     6341         * @augments Backbone.View
     6342         */
     6343        media.view.Spinner = media.View.extend({
     6344                tagName:   'span',
     6345                className: 'spinner',
     6346
     6347                show: function() {
     6348                        this.$el.show();
     6349                        return this;
     6350                },
     6351
     6352                hide: function() {
     6353                        this.$el.hide();
     6354                        return this;
     6355                }
     6356        });
    63156357}(jQuery, _));