WordPress.org
Get WordPress

Make WordPress Core

Ticket #39859: 39859.4.diff

File 39859.4.diff, 5.6 KB (added by adamsilverstein, 7 years ago)

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

    diff --git src/wp-includes/js/media-views.js src/wp-includes/js/media-views.js
index 60ad65ee48..da23cb4629 100644
    Modal = wp.media.View.extend(/** @lends wp.media.view.Modal.prototype */{ 
    42484248        tagName:  'div',
    42494249        template: wp.template('media-modal'),
    42504250
    4251         attributes: {
    4252                 tabindex: 0
    4253         },
    4254 
    42554251        events: {
    42564252                'click .media-modal-backdrop, .media-modal-close': 'escapeHandler',
    42574253                'keydown': 'keydown'
    … … Modal = wp.media.View.extend(/** @lends wp.media.view.Modal.prototype */{ 
    42634259                _.defaults( this.options, {
    42644260                        container: document.body,
    42654261                        title:     '',
    4266                         propagate: true,
    4267                         freeze:    true
     4262                        propagate: true
    42684263                });
    42694264
    42704265                this.focusManager = new wp.media.view.FocusManager({
    … … Modal = wp.media.View.extend(/** @lends wp.media.view.Modal.prototype */{ 
    43194314         */
    43204315        open: function() {
    43214316                var $el = this.$el,
    4322                         options = this.options,
    43234317                        mceEditor;
    43244318
    43254319                if ( $el.is(':visible') ) {
    … … Modal = wp.media.View.extend(/** @lends wp.media.view.Modal.prototype */{ 
    43324326                        this.attach();
    43334327                }
    43344328
    4335                 // If the `freeze` option is set, record the window's scroll position.
    4336                 if ( options.freeze ) {
    4337                         this._freeze = {
    4338                                 scrollTop: $( window ).scrollTop()
    4339                         };
    4340                 }
    4341 
    43424329                // Disable page scrolling.
    43434330                $( 'body' ).addClass( 'modal-open' );
    43444331
    … … Modal = wp.media.View.extend(/** @lends wp.media.view.Modal.prototype */{ 
    43464333
    43474334                // Try to close the onscreen keyboard
    43484335                if ( 'ontouchend' in document ) {
    4349                         if ( ( mceEditor = window.tinymce && window.tinymce.activeEditor )  && ! mceEditor.isHidden() && mceEditor.iframeElement ) {
     4336                        if ( ( mceEditor = window.tinymce && window.tinymce.activeEditor ) && ! mceEditor.isHidden() && mceEditor.iframeElement ) {
    43504337                                mceEditor.iframeElement.focus();
    43514338                                mceEditor.iframeElement.blur();
    43524339
    … … Modal = wp.media.View.extend(/** @lends wp.media.view.Modal.prototype */{ 
    43664353         * @returns {wp.media.view.Modal} Returns itself to allow chaining
    43674354         */
    43684355        close: function( options ) {
    4369                 var freeze = this._freeze;
    4370 
    43714356                if ( ! this.views.attached || ! this.$el.is(':visible') ) {
    43724357                        return this;
    43734358                }
    … … Modal = wp.media.View.extend(/** @lends wp.media.view.Modal.prototype */{ 
    43874372
    43884373                this.propagate('close');
    43894374
    4390                 // If the `freeze` option is set, restore the container's scroll position.
    4391                 if ( freeze ) {
    4392                         $( window ).scrollTop( freeze.scrollTop );
    4393                 }
    4394 
    43954375                if ( options && options.escape ) {
    43964376                        this.propagate('escape');
    43974377                }

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

    diff --git src/wp-includes/js/media/views/modal.js src/wp-includes/js/media/views/modal.js
index f31a59d6d4..cfe11ec941 100644
    Modal = wp.media.View.extend(/** @lends wp.media.view.Modal.prototype */{ 
    1717        tagName:  'div',
    1818        template: wp.template('media-modal'),
    1919
    20         attributes: {
    21                 tabindex: 0
    22         },
    23 
    2420        events: {
    2521                'click .media-modal-backdrop, .media-modal-close': 'escapeHandler',
    2622                'keydown': 'keydown'
    … … Modal = wp.media.View.extend(/** @lends wp.media.view.Modal.prototype */{ 
    3228                _.defaults( this.options, {
    3329                        container: document.body,
    3430                        title:     '',
    35                         propagate: true,
    36                         freeze:    true
     31                        propagate: true
    3732                });
    3833
    3934                this.focusManager = new wp.media.view.FocusManager({
    … … Modal = wp.media.View.extend(/** @lends wp.media.view.Modal.prototype */{ 
    8883         */
    8984        open: function() {
    9085                var $el = this.$el,
    91                         options = this.options,
    9286                        mceEditor;
    9387
    9488                if ( $el.is(':visible') ) {
    … … Modal = wp.media.View.extend(/** @lends wp.media.view.Modal.prototype */{ 
    10195                        this.attach();
    10296                }
    10397
    104                 // If the `freeze` option is set, record the window's scroll position.
    105                 if ( options.freeze ) {
    106                         this._freeze = {
    107                                 scrollTop: $( window ).scrollTop()
    108                         };
    109                 }
    110 
    11198                // Disable page scrolling.
    11299                $( 'body' ).addClass( 'modal-open' );
    113100
    … … Modal = wp.media.View.extend(/** @lends wp.media.view.Modal.prototype */{ 
    115102
    116103                // Try to close the onscreen keyboard
    117104                if ( 'ontouchend' in document ) {
    118                         if ( ( mceEditor = window.tinymce && window.tinymce.activeEditor )  && ! mceEditor.isHidden() && mceEditor.iframeElement ) {
     105                        if ( ( mceEditor = window.tinymce && window.tinymce.activeEditor ) && ! mceEditor.isHidden() && mceEditor.iframeElement ) {
    119106                                mceEditor.iframeElement.focus();
    120107                                mceEditor.iframeElement.blur();
    121108
    … … Modal = wp.media.View.extend(/** @lends wp.media.view.Modal.prototype */{ 
    135122         * @returns {wp.media.view.Modal} Returns itself to allow chaining
    136123         */
    137124        close: function( options ) {
    138                 var freeze = this._freeze;
    139 
    140125                if ( ! this.views.attached || ! this.$el.is(':visible') ) {
    141126                        return this;
    142127                }
    … … Modal = wp.media.View.extend(/** @lends wp.media.view.Modal.prototype */{ 
    156141
    157142                this.propagate('close');
    158143
    159                 // If the `freeze` option is set, restore the container's scroll position.
    160                 if ( freeze ) {
    161                         $( window ).scrollTop( freeze.scrollTop );
    162                 }
    163 
    164144                if ( options && options.escape ) {
    165145                        this.propagate('escape');
    166146                }

  • src/wp-includes/media-template.php 

    diff --git src/wp-includes/media-template.php src/wp-includes/media-template.php
index 4f2f302e0e..0dbba580f4 100644
    function wp_print_media_templates() { 
    182182        </script>
    183183
    184184        <script type="text/html" id="tmpl-media-modal">
    185                 <div class="<?php echo $class; ?>">
     185                <div tabindex="0" class="<?php echo $class; ?>">
    186186                        <button type="button" class="media-modal-close"><span class="media-modal-icon"><span class="screen-reader-text"><?php _e( 'Close media panel' ); ?></span></span></button>
    187187                        <div class="media-modal-content"></div>
    188188                </div>