WordPress.org

Make WordPress Core

Ticket #25439: 25439.2.diff

File 25439.2.diff, 6.5 KB (added by westonruter, 5 years ago)

Changes: Show AYS dialog if customizer dirty and returning to opener page. Cleanup formatting for jshint. New commits: https://github.com/x-team/wordpress-develop/compare/f21097b40e32ee03c60f676f3e8d973151cb52e0...0337bd8537426de554a03807003eec734fe895a6 Amended PR: https://github.com/x-team/wordpress-develop/pull/23

  • src/wp-admin/js/customize-controls.js

    diff --git src/wp-admin/js/customize-controls.js src/wp-admin/js/customize-controls.js
    index 4bfc252..a7a6655 100644
     
    11281128                        });
    11291129                });
    11301130
     1131                // Prompt user with AYS dialog if leaving the Customizer with unsaved changes
     1132                $( window ).on( 'beforeunload', function () {
     1133                        if ( ! api.state( 'saved' )() ) {
     1134                                return api.l10n.saveAlert;
     1135                        }
     1136                } );
     1137
    11311138                // Pass events through to the parent.
    1132                 api.bind( 'saved', function() {
    1133                         parent.send( 'saved' );
    1134                 });
     1139                $.each( [ 'saved', 'change' ], function ( i, event ) {
     1140                        api.bind( event, function() {
     1141                                parent.send( event );
     1142                        });
     1143                } );
    11351144
    11361145                // When activated, let the loader handle redirecting the page.
    11371146                // If no loader exists, redirect the page ourselves (if a url exists).
  • src/wp-includes/js/customize-loader.js

    diff --git src/wp-includes/js/customize-loader.js src/wp-includes/js/customize-loader.js
    index cccf71a..98bb32f 100644
     
    1 /* global _wpCustomizeLoaderSettings */
     1/* global _wpCustomizeLoaderSettings, confirm */
    22window.wp = window.wp || {};
    33
    44(function( exports, $ ){
    window.wp = window.wp || {}; 
    3636                        });
    3737
    3838                        // Add navigation listeners.
    39                         if ( $.support.history )
     39                        if ( $.support.history ) {
    4040                                this.window.on( 'popstate', Loader.popstate );
     41                        }
    4142
    4243                        if ( $.support.hashchange ) {
    4344                                this.window.on( 'hashchange', Loader.hashchange );
    window.wp = window.wp || {}; 
    4748
    4849                popstate: function( e ) {
    4950                        var state = e.originalEvent.state;
    50                         if ( state && state.customize )
     51                        if ( state && state.customize ) {
    5152                                Loader.open( state.customize );
    52                         else if ( Loader.active )
     53                        } else if ( Loader.active ) {
    5354                                Loader.close();
     55                        }
    5456                },
    5557
    5658                hashchange: function() {
    5759                        var hash = window.location.toString().split('#')[1];
    5860
    59                         if ( hash && 0 === hash.indexOf( 'wp_customize=on' ) )
     61                        if ( hash && 0 === hash.indexOf( 'wp_customize=on' ) ) {
    6062                                Loader.open( Loader.settings.url + '?' + hash );
     63                        }
    6164
    62                         if ( ! hash && ! $.support.history )
     65                        if ( ! hash && ! $.support.history ){
    6366                                Loader.close();
     67                        }
     68                },
     69
     70                beforeunload: function () {
     71                        if ( ! Loader.saved() ) {
     72                                return Loader.settings.l10n.saveAlert;
     73                        }
    6474                },
    6575
    6676                open: function( src ) {
    67                         var hash;
    6877
    69                         if ( this.active )
     78                        if ( this.active ) {
    7079                                return;
     80                        }
    7181
    7282                        // Load the full page on mobile devices.
    73                         if ( Loader.settings.browser.mobile )
     83                        if ( Loader.settings.browser.mobile ) {
    7484                                return window.location = src;
     85                        }
    7586
    7687                        this.active = true;
    7788                        this.body.addClass('customize-loading');
    7889
     90                        // Dirty state of customizer in iframe
     91                        this.saved = new api.Value( true );
     92
    7993                        this.iframe = $( '<iframe />', { src: src }).appendTo( this.element );
    8094                        this.iframe.one( 'load', this.loaded );
    8195
    window.wp = window.wp || {}; 
    92106                        });
    93107
    94108                        this.messenger.bind( 'close', function() {
    95                                 if ( $.support.history )
     109                                if ( $.support.history ) {
    96110                                        history.back();
    97                                 else if ( $.support.hashchange )
     111                                } else if ( $.support.hashchange ) {
    98112                                        window.location.hash = '';
    99                                 else
     113                                } else {
    100114                                        Loader.close();
    101                         });
     115                                }
     116                        } );
     117
     118                        // Prompt AYS dialog when navigating away
     119                        $( window ).on( 'beforeunload', this.beforeunload );
    102120
    103121                        this.messenger.bind( 'activated', function( location ) {
    104                                 if ( location )
     122                                if ( location ) {
    105123                                        window.location = location;
     124                                }
    106125                        });
    107126
    108                         hash = src.split('?')[1];
     127                        this.messenger.bind( 'saved', function () {
     128                                Loader.saved( true );
     129                        } );
     130                        this.messenger.bind( 'change', function () {
     131                                Loader.saved( false );
     132                        } );
     133
     134                        this.pushState( src );
     135
     136                        this.trigger( 'open' );
     137                },
     138
     139                pushState: function ( src ) {
     140                        var hash;
    109141
    110142                        // Ensure we don't call pushState if the user hit the forward button.
    111                         if ( $.support.history && window.location.href !== src )
     143                        if ( $.support.history && window.location.href !== src ) {
    112144                                history.pushState( { customize: src }, '', src );
    113                         else if ( ! $.support.history && $.support.hashchange && hash )
     145                        } else if ( ! $.support.history && $.support.hashchange && hash ) {
     146                                hash = src.split( '?' )[1];
    114147                                window.location.hash = 'wp_customize=on&' + hash;
    115 
    116                         this.trigger( 'open' );
     148                        }
    117149                },
    118150
    119151                opened: function() {
    window.wp = window.wp || {}; 
    121153                },
    122154
    123155                close: function() {
    124                         if ( ! this.active )
     156                        if ( ! this.active ) {
     157                                return;
     158                        }
     159
     160                        // Display AYS dialog if customizer is dirty
     161                        if ( ! this.saved() && ! confirm( Loader.settings.l10n.saveAlert ) ) {
     162                                // Go forward since Customizer is exited by history.back()
     163                                history.forward();
    125164                                return;
     165                        }
     166
    126167                        this.active = false;
    127168
    128169                        this.trigger( 'close' );
    129170
    130171                        // Return focus to link that was originally clicked.
    131                         if ( this.link )
     172                        if ( this.link ) {
    132173                                this.link.focus();
     174                        }
    133175                },
    134176
    135177                closed: function() {
    window.wp = window.wp || {}; 
    137179                        Loader.messenger.destroy();
    138180                        Loader.iframe    = null;
    139181                        Loader.messenger = null;
     182                        Loader.saved     = null;
    140183                        Loader.body.removeClass( 'customize-active full-overlay-active' ).removeClass( 'customize-loading' );
     184                        $( window ).off( 'beforeunload', Loader.beforeunload );
    141185                },
    142186
    143187                loaded: function() {
  • src/wp-includes/script-loader.php

    diff --git src/wp-includes/script-loader.php src/wp-includes/script-loader.php
    index df42718..c0d42b6 100644
    function wp_default_scripts( &$scripts ) { 
    381381        did_action( 'init' ) && $scripts->localize( 'customize-controls', '_wpCustomizeControlsL10n', array(
    382382                'activate'  => __( 'Save &amp; Activate' ),
    383383                'save'      => __( 'Save &amp; Publish' ),
     384                'saveAlert' => __( 'The changes you made will be lost if you navigate away from this page.' ),
    384385                'saved'     => __( 'Saved' ),
    385386                'cancel'    => __( 'Cancel' ),
    386387                'close'     => __( 'Close' ),
  • src/wp-includes/theme.php

    diff --git src/wp-includes/theme.php src/wp-includes/theme.php
    index 40acf6b..d882463 100644
    function _wp_customize_loader_settings() { 
    18741874                'url'           => esc_url( admin_url( 'customize.php' ) ),
    18751875                'isCrossDomain' => $cross_domain,
    18761876                'browser'       => $browser,
     1877                'l10n'          => array(
     1878                        'saveAlert' => __( 'The changes you made will be lost if you navigate away from this page.' ),
     1879                ),
    18771880        );
    18781881
    18791882        $script = 'var _wpCustomizeLoaderSettings = ' . json_encode( $settings ) . ';';