Ticket #7863: 7863.diff

wp-includes/script-loader.php

@@ -222 +222 @@
                         'saveText' => attribute_escape(__('Save »')),
                         'confirmText' => __("Are you sure you want to delete the file '%title%'?\nClick ok to delete or cancel to go back.")
                 ) );
-                $scripts->add( 'admin-widgets', '/wp-admin/js/widgets.js', array( 'interface' ), '20080503' );
+                $scripts->add( 'admin-widgets', '/wp-admin/js/widgets.js', array( 'interface' ), '20081010' );
                 $scripts->localize( 'admin-widgets', 'widgetsL10n', array(
                         'add' => __('Add'),
                         'edit' => __('Edit'),
                         'cancel' => __('Cancel'),
+                        'lameReminder' => __('Remember to click the "Save Changes" button at the bottom of the Current Widgets column after you\'re all done!'),
+                        'lamerReminder' => __("You're about to leave without having saved your changes!")
                 ));
 
                 $scripts->add( 'word-count', '/wp-admin/js/word-count.js', array( 'jquery' ), '20080423' );

wp-admin/includes/widgets.php

@@ -305 +305 @@
 
                                 <?php if ( $control ) : ?>
 
-                                <a class="widget-action widget-control-save hide-if-no-js edit alignleft" href="#save:<?php echo $id_format; ?>"><?php _e('Change'); ?></a>
+                                <a class="widget-action widget-control-save hide-if-no-js edit alignleft" href="#save:<?php echo $id_format; ?>"><?php _e('Done'); ?></a>
 
                                 <?php endif; ?>
 

wp-admin/js/widgets.js

@@ -1 +1 @@
 jQuery(function($) {
         $('.noscript-action').remove();
 
+        var reminded = false;
+        var lameWidgetReminder = function() {
+                if ( reminded )
+                        return;
+                window.onbeforeunload = function () { return widgetsL10n.lamerReminder };
+                $('div.wrap:first').prepend( '<div class="updated" style="display:none"><p>' + widgetsL10n.lameReminder + '</p></div>' ).children( ':first' ).slideDown();
+                $('#current-widgets .submit input[name=save-widgets]').css( 'background-color', '#ffffe0' ).click( function() {
+                        window.onbeforeunload = null;
+                } );
+                reminded = true;
+        };
+
         var increment = 1;
 
         // Open or close widget control form
@@ -84 +96 @@
                 var n = parseInt( $('#widget-count').text(), 10 ) + 1;
                 $('#widget-count').text( n.toString() )
 
+                lameWidgetReminder();
                 return false;
         };
 
@@ -96 +109 @@
 
                 // onclick for save links
                 $('a.widget-control-save', context).click( function() {
+                        lameWidgetReminder();
                         toggleWidget( $(this).parents('li:first'), false ).blur()
                         return false;
                 } );