WordPress.org

Make WordPress Core

Ticket #22613: 22613-option-b.diff

File 22613-option-b.diff, 3.2 KB (added by lessbloat, 5 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%;