Changeset 24743

Timestamp:

07/18/2013 09:33:59 PM (13 years ago)

Author:

nacin

Message:

Better 'Connection lost' notice that includes an indication of activity. fixes #24695.

Location:

trunk

Files:

• wp-admin/css/wp-admin.css (modified) (1 diff)
• wp-admin/edit-form-advanced.php (modified) (1 diff)
• wp-includes/js/autosave.js (modified) (1 diff)

trunk/wp-admin/css/wp-admin.css

@@ -2994 +2994 @@
 }
 
+#lost-connection-notice .spinner {
+    display: block;
+    float: left;
+    margin: 0 5px 0 0;
+}
+
+.rtl #lost-connection-notice .spinner {
+    float: right;
+    margin: 0 0 0 5px;
+}
+
 #titlediv {
     position: relative;

trunk/wp-admin/edit-form-advanced.php

@@ -314 +314 @@
 <?php endif; ?>
 <div id="lost-connection-notice" class="error hidden">
-    <p><?php _e("You have lost your connection with the server, and saving has been disabled. This message will vanish once you've reconnected."); ?></p>
+    <p><span class="spinner"></span> <?php _e( '<strong>Connection lost.</strong> Saving has been disabled until you&#8217;re reconnected.' ); ?>
+    <span class="hide-if-no-sessionstorage"><?php _e( 'We&#8217;re backing up this post in your browser, just in case.' ); ?></span>
+    </p>
 </div>
 

trunk/wp-includes/js/autosave.js

@@ -145 +145 @@
     $(document).on('heartbeat-connection-lost.autosave', function( e, error ) {
         if ( 'timeout' === error ) {
-            $('#lost-connection-notice').show();
+            var notice = $('#lost-connection-notice');
+            if ( ! wp.autosave.local.hasStorage ) {
+                notice.find('.hide-if-no-sessionstorage').hide();
+            }
+            notice.show();
             autosave_disable_buttons();
         }