WordPress.org

Make WordPress Core

Changeset 45172


Ignore:
Timestamp:
04/12/2019 06:07:47 PM (3 years ago)
Author:
azaozz
Message:

TinyMCE: Fix keyboard navigation when the dfw button is present but hidden. Ensure that button is added last in the DOM to match where it appears visually.

Props afercia, azaozz.
Fixes #46640.

Location:
trunk/src
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/js/_enqueues/wp/editor/dfw.js

    r45127 r45172  
    14741474                    var button = this;
    14751475
     1476                    editor.on( 'init', function() {
     1477                        if ( button.disabled() ) {
     1478                            button.hide();
     1479                        }
     1480                    } );
     1481
    14761482                    $document
    14771483                    .on( 'dfw-activate.focus', function() {
    14781484                        button.disabled( false );
     1485                        button.show();
    14791486                    } )
    14801487                    .on( 'dfw-deactivate.focus', function() {
    14811488                        button.disabled( true );
     1489                        button.hide();
    14821490                    } )
    14831491                    .on( 'dfw-on.focus', function() {
  • trunk/src/wp-admin/css/edit.css

    r44963 r45172  
    12741274-------------------------------------------------------------- */
    12751275
    1276 #wp-content-wrap .mce-wp-dfw,
    12771276#qt_content_dfw {
    12781277    display: none;
    12791278}
    12801279
    1281 .wp-editor-expand #wp-content-wrap .mce-wp-dfw,
    12821280.wp-editor-expand #qt_content_dfw {
    12831281    display: inline-block;
  • trunk/src/wp-includes/class-wp-editor.php

    r45066 r45172  
    588588                if ( ! wp_is_mobile() ) {
    589589                    if ( $set['_content_editor_dfw'] ) {
     590                        $mce_buttons[] = 'wp_adv';
    590591                        $mce_buttons[] = 'dfw';
    591592                    } else {
    592593                        $mce_buttons[] = 'fullscreen';
     594                        $mce_buttons[] = 'wp_adv';
    593595                    }
     596                } else {
     597                    $mce_buttons[] = 'wp_adv';
    594598                }
    595 
    596                 $mce_buttons[] = 'wp_adv';
    597599
    598600                /**
  • trunk/src/wp-includes/css/editor.css

    r44791 r45172  
    12701270    top: 0;
    12711271    right: 0;
     1272}
     1273
     1274.mce-toolbar .mce-btn-group .mce-btn.mce-wp-dfw {
     1275    margin: 7px 7px 0 0;
     1276}
     1277
     1278.qt-dfw {
    12721279    margin: 5px 5px 0 0;
    12731280}
Note: See TracChangeset for help on using the changeset viewer.