Changeset 4466

Timestamp:

01/11/12 15:29:25 (4 months ago)

Author:

tboutell

Message:

* Date widget sends change() events to the widgets it is enhancing
* timepicker2 (and therefore our time widget) sends change() events to the the widgets it is enhancing
* apostrophe.onBeforeUnload is called from globalJavascripts. If there is an edit view visible, or there is an event or blog post metadata form claiming to have unsaved changes, and the user tries to leave the page, they are warned that they may have unsaved data

Location:

plugins/apostrophePlugin/branches/1.5

Files:

• lib/widget/aWidgetFormJQueryDate.class.php (modified) (2 diffs)
• modules/a/templates/_globalJavascripts.php (modified) (1 diff)
• web/js/a.js (modified) (1 diff)
• web/js/timepicker.js (modified) (1 diff)

plugins/apostrophePlugin/branches/1.5/lib/widget/aWidgetFormJQueryDate.class.php

@@ -125 +125 @@
     \$(day).val('');
     \$(year).val('');
+    \$(month).change();
+    \$(day).change();
+    \$(year).change();
     return;
   }
@@ -132 +135 @@
   \$(day).val(components[2].replace(/^0+/, ''));
   \$(year).val(components[3].replace(/^0+/, ''));
+  \$(month).change();
+  \$(day).change();
+  \$(year).change();
   
   // Something we can bind to update other fields 

plugins/apostrophePlugin/branches/1.5/modules/a/templates/_globalJavascripts.php

@@ -21 +21 @@
 <?php // A handful of fundamental improvements like a-autosubmit for anchor buttons ?>
 <?php a_js_call('apostrophe.smartCSS()') ?>
+<?php // Warn the user if they try to leave the page and it has open slot editors on it ?>
+<?php a_js_call('apostrophe.onBeforeUnload()') ?>
 <?php // End of body - time to emit all of the queued JS as one script block ?>
 

plugins/apostrophePlugin/branches/1.5/web/js/a.js

@@ -2274 +2274 @@
         };
 
+  this.onBeforeUnload = function()
+  {
+    $(window).unbind('beforeunload.apostrophe').bind('beforeunload.apostrophe', function() {
+      // Event first since it gets both classes
+      if ($('.a-admin-event-form form').data('changed'))
+      {
+        return "You have not saved your changes to the settings for this event in the form at left. Any edits you have made will be lost if you do not save them first before leaving the page.";
+      }
+      if ($('.a-admin-blog-post-form form').data('changed'))
+      {
+        return "You have not saved your changes to the settings for this blog post in the form at left. Any edits you have made will be lost if you do not save them first before leaving the page.";
+      }
+      if ($('.a-slot.a-editing').length)
+      {
+        return "You are still editing content. Any edits you have made will be lost if you do not save them first before leaving the page.";
+      }
+    });
+  }
+  
         // Breaks the url into a stem (everything before the query, inclusive of the ?), a query
         // (the encoded query string), and queryData (the query string parsed into an object)

plugins/apostrophePlugin/branches/1.5/web/js/timepicker.js

@@ -206 +206 @@
                                 $(inputs[0]).val(time.hours);
                                 $(inputs[1]).val(time.minutes);
+                                $(inputs[0]).change();
+                                $(inputs[1]).change();
                         }
                 }