WordPress.org
Get WordPress

Make WordPress Core

Ticket #30725: 30725.5.diff

File 30725.5.diff, 5.5 KB (added by obenland, 10 years ago)

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

     
    14941494        background: url(../images/spinner.gif) no-repeat;
    14951495        -webkit-background-size: 20px 20px;
    14961496        background-size: 20px 20px;
    1497         display: none;
    14981497        opacity: 0.7;
    14991498        filter: alpha(opacity=70);
    15001499        width: 20px;

  • src/wp-includes/js/media/views/attachments/browser.js

     
    306306                }
    307307
    308308                if ( ! this.collection.length ) {
    309                         this.toolbar.get( 'spinner' ).show();
     309                        this.toolbar.get( 'spinner' ).$el.addClass( 'is-active' );
    310310                        this.dfd = this.collection.more().done( function() {
    311311                                if ( ! view.collection.length ) {
    312312                                        noItemsView.$el.removeClass( 'hidden' );
    313313                                } else {
    314314                                        noItemsView.$el.addClass( 'hidden' );
    315315                                }
    316                                 view.toolbar.get( 'spinner' ).hide();
     316                                view.toolbar.get( 'spinner' ).$el.removeClass( 'is-active' );
    317317                        } );
    318318                } else {
    319319                        noItemsView.$el.addClass( 'hidden' );
    320                         view.toolbar.get( 'spinner' ).hide();
     320                        view.toolbar.get( 'spinner' ).$el.removeClass( 'is-active' );
    321321                }
    322322        },
    323323

  • src/wp-includes/js/media/views/attachments.js

     
    284284
    285285                // Show the spinner only if we are close to the bottom.
    286286                if ( el.scrollHeight - ( scrollTop + el.clientHeight ) < el.clientHeight / 3 ) {
    287                         toolbar.get('spinner').show();
     287                        toolbar.get('spinner').$el.addClass( 'is-active' );
    288288                }
    289289
    290290                if ( el.scrollHeight < scrollTop + ( el.clientHeight * this.options.refreshThreshold ) ) {
    291291                        this.collection.more().done(function() {
    292292                                view.scroll();
    293                                 toolbar.get('spinner').hide();
     293                                toolbar.get('spinner').$el.removeClass( 'is-active' );
    294294                        });
    295295                }
    296296        }

  • src/wp-includes/js/media/views/embed/link.js

     
    3434                        return;
    3535                }
    3636
    37                 this.spinner.show();
     37                this.spinner.addClass( 'is-active' );
    3838
    3939                this.fetch();
    4040        }, 600 ),
    … …  
    9595                        this.model.unset( 'width', opts );
    9696                }
    9797
    98                 this.spinner.hide();
     98                this.spinner.removeClass( 'is-active' );
    9999
    100100                this.$('.embed-container').show().find('.embed-preview').html( html );
    101101        }

  • src/wp-includes/js/media/views/spinner.js

     
    1717        show: function() {
    1818                if ( ! this.spinnerTimeout ) {
    1919                        this.spinnerTimeout = _.delay(function( $el ) {
    20                                 $el.show();
     20                                $el.addClass( 'is-active' );
    2121                        }, this.delay, this.$el );
    2222                }
    2323
    … …  
    2525        },
    2626
    2727        hide: function() {
    28                 this.$el.hide();
     28                this.$el.removeClass( 'is-active' );
    2929                this.spinnerTimeout = clearTimeout( this.spinnerTimeout );
    3030
    3131                return this;

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

     
    36553655
    36563656                // Show the spinner only if we are close to the bottom.
    36573657                if ( el.scrollHeight - ( scrollTop + el.clientHeight ) < el.clientHeight / 3 ) {
    3658                         toolbar.get('spinner').show();
     3658                        toolbar.get('spinner').$el.addClass( 'is-active' );
    36593659                }
    36603660
    36613661                if ( el.scrollHeight < scrollTop + ( el.clientHeight * this.options.refreshThreshold ) ) {
    36623662                        this.collection.more().done(function() {
    36633663                                view.scroll();
    3664                                 toolbar.get('spinner').hide();
     3664                                toolbar.get('spinner').$el.removeClass( 'is-active' );
    36653665                        });
    36663666                }
    36673667        }
    … …  
    39783978                }
    39793979
    39803980                if ( ! this.collection.length ) {
    3981                         this.toolbar.get( 'spinner' ).show();
     3981                        this.toolbar.get( 'spinner' ).$el.addClass( 'is-active' );
    39823982                        this.dfd = this.collection.more().done( function() {
    39833983                                if ( ! view.collection.length ) {
    39843984                                        noItemsView.$el.removeClass( 'hidden' );
    39853985                                } else {
    39863986                                        noItemsView.$el.addClass( 'hidden' );
    39873987                                }
    3988                                 view.toolbar.get( 'spinner' ).hide();
     3988                                view.toolbar.get( 'spinner').$el.removeClass( 'is-active' );
    39893989                        } );
    39903990                } else {
    39913991                        noItemsView.$el.addClass( 'hidden' );
    3992                         view.toolbar.get( 'spinner' ).hide();
     3992                        view.toolbar.get( 'spinner').$el.removeClass( 'is-active' );
    39933993                }
    39943994        },
    39953995
    … …  
    45464546                        return;
    45474547                }
    45484548
    4549                 this.spinner.show();
     4549                this.spinner.addClass( 'is-active' );
    45504550
    45514551                this.fetch();
    45524552        }, 600 ),
    … …  
    46074607                        this.model.unset( 'width', opts );
    46084608                }
    46094609
    4610                 this.spinner.hide();
     4610                this.spinner.removeClass( 'is-active' );
    46114611
    46124612                this.$('.embed-container').show().find('.embed-preview').html( html );
    46134613        }
    … …  
    74637463        show: function() {
    74647464                if ( ! this.spinnerTimeout ) {
    74657465                        this.spinnerTimeout = _.delay(function( $el ) {
    7466                                 $el.show();
     7466                                $el.addClass( 'is-active' );
    74677467                        }, this.delay, this.$el );
    74687468                }
    74697469
    … …  
    74717471        },
    74727472
    74737473        hide: function() {
    7474                 this.$el.hide();
     7474                this.$el.removeClass( 'is-active' );
    74757475                this.spinnerTimeout = clearTimeout( this.spinnerTimeout );
    74767476
    74777477                return this;