WordPress.org
Get WordPress

Make WordPress Core

Changeset 27038


Ignore:
Timestamp:
01/27/2014 12:27:42 AM (11 years ago)
Author:
azaozz
Message:

Autosave:

  • Move the 'Saving post' and 'Draft saved at...' strings from autosaveL10n to postL10n as they are used only there.
  • Use the custom jQuery events 'before-autosave' and 'after-autosave' to show these messages.
  • Separate autosave.suspend() for local and server so local autosaves can continue while server autosaves are suspended.
  • Remove the recently added autosave.server.disable() and use autosave.server.suspend() instead.
  • Bring back .button.disabled, button-primary.disabled and use .disabled to prevent multiple form submissions.

See #25272.

Location:
trunk/src
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-admin/js/post.js

    r27035 r27038  
    1 /* global postL10n, ajaxurl, wpAjax, setPostThumbnailL10n, postboxes, pagenow, tinymce, alert, deleteUserSetting, getUserSetting, setUserSetting */
    2 /* global theList:true, theExtraList:true, autosave:true */
    3 
    4 var tagBox, commentsBox, WPSetThumbnailHTML, WPSetThumbnailID, WPRemoveThumbnail, wptitlehint;
     1/* global postL10n, ajaxurl, wpAjax, setPostThumbnailL10n, postboxes, pagenow, tinymce, alert, deleteUserSetting */
     2/* global theList:true, theExtraList:true, getUserSetting, setUserSetting */
     3
     4var tagBox, commentsBox, WPSetThumbnailHTML, WPSetThumbnailID, WPRemoveThumbnail, wptitlehint, makeSlugeditClickable, editPermalink;
    55// Back-compat: prevent fatal errors
    66makeSlugeditClickable = editPermalink = function(){};
     7
     8window.wp = window.wp || {};
    79
    810// return an array with any duplicate, whitespace or values removed
    … …  
    284286
    285287            if ( wrap.length && ! wrap.is(':visible') ) {
    286                 if ( typeof wp != 'undefined' && wp.autosave ) {
     288                if ( wp.autosave ) {
    287289                    // Save the latest changes and disable
    288290                    $(document).one( 'heartbeat-tick', function() {
    289                         wp.autosave.server.disable();
     291                        wp.autosave.server.suspend();
    290292                        wrap.removeClass('saving').addClass('saved');
    291293                        $(window).off( 'beforeunload.edit-post' );
    … …  
    314316                action: 'sample-permalink',
    315317                post_id: $('#post_ID').val(),
    316                 new_title: typeof fullscreen != 'undefined' && fullscreen.settings.visible ? $('#wp-fullscreen-title').val() : $('#title').val(),
     318                new_title: $('#title').val(),
    317319                samplepermalinknonce: $('#samplepermalinknonce').val()
    318320            },
    … …  
    369371
    370372jQuery(document).ready( function($) {
    371     var stamp, visibility, $submitButtons,
     373    var stamp, visibility, $submitButtons, updateVisibility, updateText,
    372374        sticky = '',
    373375        last = 0,
    … …  
    400402
    401403    // Set the heartbeat interval to 15 sec. if post lock dialogs are enabled
    402     if ( typeof wp !== 'undefined' && wp.heartbeat && $('#post-lock-dialog').length ) {
     404    if ( wp.heartbeat && $('#post-lock-dialog').length ) {
    403405        wp.heartbeat.interval( 15 );
    404406    }
    … …  
    408410        var $button = $(this);
    409411
    410         if ( $button.hasClass('button-disabled') ) {
     412        if ( $button.hasClass('disabled') ) {
    411413            event.preventDefault();
    412414            return;
    … …  
    424426            }
    425427
    426             if ( typeof wp != 'undefined' && wp.autosave ) {
    427                 wp.autosave.server.disable();
     428            // Stop autosave
     429            if ( wp.autosave ) {
     430                wp.autosave.server.suspend();
    428431            }
    429432
    … …  
    431434            $(window).off( 'beforeunload.edit-post' );
    432435
    433             $submitButtons.addClass( 'button-disabled' );
     436            $submitButtons.addClass( 'disabled' );
    434437
    435438            if ( $button.attr('id') === 'publish' ) {
    … …  
    448451            target = $this.attr('target') || 'wp-preview',
    449452            ua = navigator.userAgent.toLowerCase();
    450        
     453
    451454        event.preventDefault();
    452455
    453         if ( $this.prop('disabled') ) {
     456        if ( $this.hasClass('disabled') ) {
    454457            return;
    455458        }
    456459
    457         if ( typeof wp != 'undefined' && wp.autosave ) {
     460        if ( wp.autosave ) {
    458461            wp.autosave.server.tempBlockSave();
    459462        }
    … …  
    497500            }
    498501
    499             if ( typeof wp != 'undefined' && wp.autosave ) {
     502            if ( wp.autosave ) {
    500503                wp.autosave.server.triggerSave();
    501504            }
    … …  
    504507
    505508    $(document).on( 'autosave-disable-buttons.edit-post', function() {
    506         $submitButtons.addClass( 'button-disabled' );
     509        $submitButtons.addClass( 'disabled' );
    507510    }).on( 'autosave-enable-buttons.edit-post', function() {
    508         if ( ! window.wp || ! window.wp.heartbeat || ! window.wp.heartbeat.hasConnectionError() ) {
    509             $submitButtons.removeClass( 'button-disabled' );
    510         }
     511        if ( ! wp.heartbeat || ! wp.heartbeat.hasConnectionError() ) {
     512            $submitButtons.removeClass( 'disabled' );
     513        }
     514    }).on( 'before-autosave.edit-post', function() {
     515        $( '.autosave-message' ).text( postL10n.savingText );
     516    }).on( 'after-autosave.edit-post', function( event, data ) {
     517        $( '.autosave-message' ).text( data.message );
    511518    });
    512519
    … …  
    514521        var editor = typeof tinymce !== 'undefined' && tinymce.get('content');
    515522
    516         if ( ( editor && ! editor.isHidden() && editor.isDirty() ) || 
    517             ( typeof wp !== 'undefined' && wp.autosave && wp.autosave.server.postChanged() ) ) {
    518 
    519             return autosaveL10n.saveAlert;
     523        if ( ( editor && ! editor.isHidden() && editor.isDirty() ) ||
     524            ( wp.autosave && wp.autosave.server.postChanged() ) ) {
     525
     526            return postL10n.saveAlert;
    520527        }
    521528    }).on( 'unload.edit-post', function( event ) {
    … …  
    818825                $timestampdiv.show();
    819826
    820                 if ( typeof wp !== 'undefined' && wp.autosave ) {
     827                if ( wp.autosave ) {
    821828                    wp.autosave.enableButtons();
    822829                }
    … …  
    849856
    850857    // permalink
     858    function editPermalink() {
     859        var i, slug_value,
     860            c = 0,
     861            e = $('#editable-post-name'),
     862            revert_e = e.html(),
     863            real_slug = $('#post_name'),
     864            revert_slug = real_slug.val(),
     865            b = $('#edit-slug-buttons'),
     866            revert_b = b.html(),
     867            full = $('#editable-post-name-full').html();
     868
     869        $('#view-post-btn').hide();
     870        b.html('<a href="#" class="save button button-small">'+postL10n.ok+'</a> <a class="cancel" href="#">'+postL10n.cancel+'</a>');
     871        b.children('.save').click(function() {
     872            var new_slug = e.children('input').val();
     873            if ( new_slug == $('#editable-post-name-full').text() ) {
     874                return $('#edit-slug-buttons .cancel').click();
     875            }
     876            $.post(ajaxurl, {
     877                action: 'sample-permalink',
     878                post_id: postId,
     879                new_slug: new_slug,
     880                new_title: $('#title').val(),
     881                samplepermalinknonce: $('#samplepermalinknonce').val()
     882            }, function(data) {
     883                var box = $('#edit-slug-box');
     884                box.html(data);
     885                if (box.hasClass('hidden')) {
     886                    box.fadeIn('fast', function () {
     887                        box.removeClass('hidden');
     888                    });
     889                }
     890                b.html(revert_b);
     891                real_slug.val(new_slug);
     892                $('#view-post-btn').show();
     893            });
     894            return false;
     895        });
     896
     897        $('#edit-slug-buttons .cancel').click(function() {
     898            $('#view-post-btn').show();
     899            e.html(revert_e);
     900            b.html(revert_b);
     901            real_slug.val(revert_slug);
     902            return false;
     903        });
     904
     905        for ( i = 0; i < full.length; ++i ) {
     906            if ( '%' == full.charAt(i) )
     907                c++;
     908        }
     909
     910        slug_value = ( c > full.length / 4 ) ? '' : full;
     911        e.html('<input type="text" id="new-post-slug" value="'+slug_value+'" />').children('input').keypress(function(e) {
     912            var key = e.keyCode || 0;
     913            // on enter, just save the new slug, don't save the post
     914            if ( 13 == key ) {
     915                b.children('.save').click();
     916                return false;
     917            }
     918            if ( 27 == key ) {
     919                b.children('.cancel').click();
     920                return false;
     921            }
     922        } ).keyup( function() {
     923            real_slug.val(this.value);
     924        }).focus();
     925    }
     926
    851927    if ( $editSlugWrap.length ) {
    852         function editPermalink() {
    853             var i, c = 0, e = $('#editable-post-name'), revert_e = e.html(), real_slug = $('#post_name'),
    854                 revert_slug = real_slug.val(), b = $('#edit-slug-buttons'), revert_b = b.html(),
    855                 full = $('#editable-post-name-full').html();
    856 
    857             $('#view-post-btn').hide();
    858             b.html('<a href="#" class="save button button-small">'+postL10n.ok+'</a> <a class="cancel" href="#">'+postL10n.cancel+'</a>');
    859             b.children('.save').click(function() {
    860                 var new_slug = e.children('input').val();
    861                 if ( new_slug == $('#editable-post-name-full').text() ) {
    862                     return $('#edit-slug-buttons .cancel').click();
    863                 }
    864                 $.post(ajaxurl, {
    865                     action: 'sample-permalink',
    866                     post_id: postId,
    867                     new_slug: new_slug,
    868                     new_title: $('#title').val(),
    869                     samplepermalinknonce: $('#samplepermalinknonce').val()
    870                 }, function(data) {
    871                     var box = $('#edit-slug-box');
    872                     box.html(data);
    873                     if (box.hasClass('hidden')) {
    874                         box.fadeIn('fast', function () {
    875                             box.removeClass('hidden');
    876                         });
    877                     }
    878                     b.html(revert_b);
    879                     real_slug.val(new_slug);
    880                     $('#view-post-btn').show();
    881                 });
    882                 return false;
    883             });
    884 
    885             $('#edit-slug-buttons .cancel').click(function() {
    886                 $('#view-post-btn').show();
    887                 e.html(revert_e);
    888                 b.html(revert_b);
    889                 real_slug.val(revert_slug);
    890                 return false;
    891             });
    892 
    893             for ( i = 0; i < full.length; ++i ) {
    894                 if ( '%' == full.charAt(i) )
    895                     c++;
    896             }
    897 
    898             slug_value = ( c > full.length / 4 ) ? '' : full;
    899             e.html('<input type="text" id="new-post-slug" value="'+slug_value+'" />').children('input').keypress(function(e) {
    900                 var key = e.keyCode || 0;
    901                 // on enter, just save the new slug, don't save the post
    902                 if ( 13 == key ) {
    903                     b.children('.save').click();
    904                     return false;
    905                 }
    906                 if ( 27 == key ) {
    907                     b.children('.cancel').click();
    908                     return false;
    909                 }
    910             } ).keyup( function() {
    911                 real_slug.val(this.value);
    912             }).focus();
    913         };
    914 
    915928        $editSlugWrap.on( 'click', function( event ) {
    916929            var $target = $( event.target );
    … …  
    972985            $textarea = $('textarea#content'),
    973986            $handle = $('#post-status-info');
    974        
     987
    975988        // No point for touch devices
    976989        if ( ! $textarea.length || 'ontouchstart' in window ) {

  • trunk/src/wp-includes/css/buttons.css

    r26993 r27038  
    166166.wp-core-ui .button[disabled],
    167167.wp-core-ui .button:disabled,
     168.wp-core-ui .button.disabled,
    168169.wp-core-ui .button-secondary[disabled],
    169170.wp-core-ui .button-secondary:disabled,
     171.wp-core-ui .button-secondary.disabled,
    170172.wp-core-ui .button-disabled {
    171173    color: #aaa !important;
    … …  
    222224.wp-core-ui .button-primary[disabled],
    223225.wp-core-ui .button-primary:disabled,
    224 .wp-core-ui .button-primary-disabled {
     226.wp-core-ui .button-primary-disabled,
     227.wp-core-ui .button-primary.disabled {
    225228    color: #94cde7 !important;
    226229    background: #298cba !important;

  • trunk/src/wp-includes/js/autosave.js

    r26995 r27038  
    77        var initialCompareString,
    88        lastTriggerSave = 0,
    9         isSuspended = false,
    109        $document = $(document);
    1110
    … …  
    8887        }
    8988
    90         function suspend() {
    91             isSuspended = true;
    92         }
    93 
    94         function resume() {
    95             isSuspended = false;
    96         }
    97 
    9889        // Autosave in localStorage
    9990        function autosaveLocal() {
    10091            var restorePostData, undoPostData, blog_id, post_id, hasStorage, intervalTimer,
    101                 lastCompareString;
     92                lastCompareString,
     93                isSuspended = false;
    10294
    10395            // Check if the browser supports sessionStorage and it's not disabled
    … …  
    195187
    196188                return setStorage( stored );
     189            }
     190
     191            function suspend() {
     192                isSuspended = true;
     193            }
     194
     195            function resume() {
     196                isSuspended = false;
    197197            }
    198198
    … …  
    416416                hasStorage: hasStorage,
    417417                getSavedPostData: getSavedPostData,
    418                 save: save
     418                save: save,
     419                suspend: suspend,
     420                resume: resume
    419421            };
    420422        }
    … …  
    422424        // Autosave on the server
    423425        function autosaveServer() {
    424             var _disabled, _blockSave, _blockSaveTimer, previousCompareString, lastCompareString,
    425                 nextRun = 0;
     426            var _blockSave, _blockSaveTimer, previousCompareString, lastCompareString,
     427                nextRun = 0,
     428                isSuspended = false;
    426429
    427430            // Block saving for the next 10 sec.
    … …  
    433436                    _blockSave = false;
    434437                }, 10000 );
     438            }
     439
     440            function suspend() {
     441                isSuspended = true;
     442            }
     443
     444            function resume() {
     445                isSuspended = false;
    435446            }
    436447
    … …  
    443454
    444455                $document.trigger( 'after-autosave', [data] );
    445                 $( '.autosave-message' ).text( data.message );
    446456                enableButtons();
    447457
    … …  
    453463
    454464            /**
    455              * Disable autosave
    456              *
    457              * Intended to run on form.submit
    458              */
    459             function disable() {
    460                 _disabled = true;
    461             }
    462 
    463             /**
    464465             * Save immediately
    465466             *
    … …  
    489490                var postData, compareString;
    490491
    491                 if ( isSuspended || _disabled || _blockSave ) {
     492                if ( isSuspended || _blockSave ) {
    492493                    return false;
    493494                }
    … …  
    517518                    .trigger( 'before-autosave', [ postData ] );
    518519
    519                 $( '.autosave-message' ).text( autosaveL10n.savingText );
    520520                postData._wpnonce = $( '#_wpnonce' ).val() || '';
    521521
    … …  
    557557
    558558            return {
    559                 disable: disable,
    560559                tempBlockSave: tempBlockSave,
    561560                triggerSave: triggerSave,
    562                 postChanged: postChanged
     561                postChanged: postChanged,
     562                suspend: suspend,
     563                resume: resume
    563564            };
    564565        }
    … …  
    585586            disableButtons: disableButtons,
    586587            enableButtons: enableButtons,
    587             suspend: suspend,
    588             resume: resume,
    589588            local: autosaveLocal(),
    590589            server: autosaveServer()

  • trunk/src/wp-includes/script-loader.php

    r27028 r27038  
    431431            'published' => __('Published'),
    432432            'comma' => _x( ',', 'tag delimiter' ),
     433            'saveAlert' => __('The changes you made will be lost if you navigate away from this page.'),
     434            'savingText' => __('Saving Draft&#8230;'),
    433435        ) );
    434436
    … …  
    654656    wp_localize_script( 'autosave', 'autosaveL10n', array(
    655657        'autosaveInterval' => AUTOSAVE_INTERVAL,
    656         'savingText' => __('Saving Draft&#8230;'),
    657         'saveAlert' => __('The changes you made will be lost if you navigate away from this page.'),
    658658        'blog_id' => get_current_blog_id(),
    659659    ) );
Note: See TracChangeset for help on using the changeset viewer.