Changeset 7813

Timestamp:

04/25/2008 12:32:29 AM (18 years ago)

Author:

ryan

Message:

Bump the autosave re-enable delay to 5 seconds, makes sure we manually re-enable the buttons on ajax success, and visually dim the disabled buttons for some UI feedback. Props mdawaffe. fixes #6707 for trunk

Location:

trunk

Files:

• wp-admin/css/colors-classic.css (modified) (1 diff)
• wp-admin/css/colors-fresh.css (modified) (1 diff)
• wp-includes/js/autosave.js (modified) (3 diffs)
• wp-includes/script-loader.php (modified) (1 diff)

trunk/wp-admin/css/colors-classic.css

@@ -191 +191 @@
         color: #07273e;
         border-color: #a3a3a3;
+}
+
+.button[disabled], .button:disabled {
+        background-color: #999;
 }
 

trunk/wp-admin/css/colors-fresh.css

@@ -187 +187 @@
         color: #246;
         border-color: #80b5d0;
+}
+
+.button[disabled], .button:disabled {
+        background-color: #999;
 }
 

trunk/wp-includes/js/autosave.js

@@ -57 +57 @@
                 var tempID = jQuery('#post_ID').val();
                 var postID = parseInt( res.responses[0].id );
-                autosave_update_post_ID( postID );
+                autosave_update_post_ID( postID ); // disabled form buttons are re-enabled here
                 if ( tempID < 0 && postID > 0) // update media buttons
                         jQuery('#media-buttons a').each(function(){
                                 this.href = this.href.replace(tempID, postID);
                         });
-        }
-        autosave_enable_buttons(); // re-enable disabled form buttons
+        } else {
+                autosave_enable_buttons(); // re-enable disabled form buttons
+        }
 }
 
@@ -79 +80 @@
                 }, function(html) {
                         jQuery('#_wpnonce').val(html);
+                        autosave_enable_buttons(); // re-enable disabled form buttons
                 });
                 jQuery('#hiddenaction').val('editpost');
@@ -128 +130 @@
 function autosave_disable_buttons() {
         jQuery("#submitpost :button:enabled, #submitpost :submit:enabled").attr('disabled', 'disabled');
-        setTimeout(autosave_enable_buttons, 1000); // Re-enable 1 sec later.  Just gives autosave a head start to avoid collisions.
+        setTimeout(autosave_enable_buttons, 5000); // Re-enable 5 sec later.  Just gives autosave a head start to avoid collisions.
 }
 

trunk/wp-includes/script-loader.php

@@ -48 +48 @@
                 ) );
 
-                $this->add( 'autosave', '/wp-includes/js/autosave.js', array('schedule', 'wp-ajax-response'), '20080422' );
+                $this->add( 'autosave', '/wp-includes/js/autosave.js', array('schedule', 'wp-ajax-response'), '20080424' );
 
                 $this->add( 'wp-ajax', '/wp-includes/js/wp-ajax.js', array('prototype'), '20070306');