WordPress.org

Make WordPress Core

Ticket #22613: 22613-option-b.diff

File 22613-option-b.diff, 3.2 KB (added by lessbloat, 3 years ago)
  • wp-includes/js/media-views.js

     
    25742574                }, 
    25752575 
    25762576                updateSetting: function( event ) { 
    2577                         var $setting = $( event.target ).closest('[data-setting]'); 
     2577                        var target = $( event.target ), 
     2578                                $setting = target.closest('[data-setting]'), 
     2579                                el = $('.' + target.attr('class')), 
     2580                                spinner = el.parent().find('.spinner'); 
    25782581 
    25792582                        if ( ! $setting.length ) 
    25802583                                return; 
     2584                         
     2585                        if ( !spinner.length ) 
     2586                                spinner = el.before('<span class="spinner"></span>').parent().find('.spinner'); 
    25812587 
    2582                         this.model.save( $setting.data('setting'), event.target.value ); 
     2588                        spinner.show(); 
     2589 
     2590                        this.model.save( $setting.data('setting'), event.target.value ).done( 
     2591                                setTimeout( function(){ 
     2592                                        spinner.hide(); 
     2593                                }, 500 ) 
     2594                        ); 
    25832595                }, 
    25842596 
    25852597                updateAll: function() { 
     
    36523664                        this.$('img').attr( 'src', this.model.get('url') ); 
    36533665                } 
    36543666        }); 
    3655 }(jQuery)); 
    3656  No newline at end of file 
     3667}(jQuery)); 
  • wp-includes/media.php

     
    12531253                'multipart'           => true, 
    12541254                'urlstream_upload'    => true, 
    12551255        ); 
    1256  
     1256         
    12571257        // Multi-file uploading doesn't currently work in iOS Safari, 
    12581258        // single-file allows the built-in camera to be used as source for images 
    12591259        if ( wp_is_mobile() ) 
     
    16781678                <# if ( 'image' === data.type ) { #> 
    16791679                        <label class="setting" data-setting="title"> 
    16801680                                <span><?php _e('Title'); ?></span> 
    1681                                 <input type="text" value="{{ data.title }}" /> 
     1681                                <input class="setting-image-title" type="text" value="{{ data.title }}" /> 
    16821682                        </label> 
    16831683                        <label class="setting" data-setting="caption"> 
    16841684                                <span><?php _e('Caption'); ?></span> 
    1685                                 <textarea 
     1685                                <textarea class="setting-image-caption" 
    16861686                                        placeholder="<?php esc_attr_e('Describe this image&hellip;'); ?>" 
    16871687                                        >{{ data.caption }}</textarea> 
    16881688                        </label> 
    16891689                        <label class="setting" data-setting="alt"> 
    16901690                                <span><?php _e('Alt Text'); ?></span> 
    1691                                 <input type="text" value="{{ data.alt }}" /> 
     1691                                <input class="setting-image-alt" type="text" value="{{ data.alt }}" /> 
    16921692                        </label> 
    16931693                <# } else { #> 
    16941694                        <label class="setting" data-setting="title"> 
    16951695                                <span><?php _e('Title'); ?></span> 
    1696                                 <input type="text" value="{{ data.title }}" 
     1696                                <input class="setting-media-title" type="text" value="{{ data.title }}" 
    16971697                                <# if ( 'video' === data.type ) { #> 
    16981698                                        placeholder="<?php esc_attr_e('Describe this video&hellip;'); ?>" 
    16991699                                <# } else if ( 'audio' === data.type ) { #> 
  • wp-includes/css/media-views.css

     
    260260        float: left; 
    261261        width: 100%; 
    262262        margin: 1px 0; 
     263        position: relative; 
    263264} 
    264265 
     266.media-sidebar .setting .spinner { 
     267        float: none; 
     268        left: 0; 
     269        min-height: 0; 
     270        position: absolute; 
     271        top: 2px; 
     272        width: auto; 
     273} 
     274 
    265275.media-sidebar .setting span { 
    266276        min-width: 30%; 
    267277        margin-right: 4%;