Changeset 39144

Timestamp:

11/04/2016 08:53:56 PM (8 years ago)

Author:

westonruter

Message:

Customize: Remove edit shortcut toggle button from preview since visibility is now linked with pane visibility.

Amends [39131].
Props westonruter, sirbrillig.
See #27403.
Fixes #38668.

Location:

trunk/src/wp-includes

Files:

• customize/class-wp-customize-selective-refresh.php (modified) (1 diff)
• js/customize-selective-refresh.js (modified) (1 diff)

trunk/src/wp-includes/customize/class-wp-customize-selective-refresh.php

@@ -189 +189 @@
                 'clickEditTitle' => __( 'Click to edit the site title.' ),
                 'clickEditMisc' => __( 'Click to edit this element.' ),
-                'editShortcutVisibilityToggle'  => __( 'Toggle edit shortcuts' ),
                 /* translators: %s: document.write() */
                 'badDocumentWrite' => sprintf( __( '%s is forbidden' ), 'document.write()' ),

trunk/src/wp-includes/js/customize-selective-refresh.js

@@ -1007 +1007 @@
 
         api.preview.bind( 'active', function() {
-            var body = $( document.body ), buttonText, $editShortcutVisibilityButton;
-
-            // Add invisible button to toggle editShortcutVisibility.
-            if ( $( '.edit-shortcut-visibility-button' ).length < 1 ) {
-                buttonText = self.data.l10n.editShortcutVisibilityToggle || 'Toggle edit shortcuts';
-                $editShortcutVisibilityButton = $( '<button type="button" class="edit-shortcut-visibility-button screen-reader-text"></button>' );
-                $editShortcutVisibilityButton.text( buttonText );
-                $editShortcutVisibilityButton.on( 'click', function() {
-                    api.selectiveRefresh.editShortcutVisibility.set( 'visible' === api.selectiveRefresh.editShortcutVisibility.get() ? 'hidden' : 'visible' );
-                } );
-                body.prepend( $editShortcutVisibilityButton );
-            }
 
             // Make all partials ready.