r13654 - in MoreFieldsAndWidgets/archetypes.referencebrowserwidget/trunk: docs src/archetypes/referencebrowserwidget/skins/referencebrowser

Author: cewing
Date: Sat May 28 00:50:40 2011
New Revision: 13654

Modified:
   MoreFieldsAndWidgets/archetypes.referencebrowserwidget/trunk/docs/CHANGES.txt
   MoreFieldsAndWidgets/archetypes.referencebrowserwidget/trunk/src/archetypes/referencebrowserwidget/skins/referencebrowser/referencebrowser.js
Log:
* set onclick handlers generated after an item is moved to return false so pages don't reload after a second move for a single item.

refs #11859




Modified: MoreFieldsAndWidgets/archetypes.referencebrowserwidget/trunk/docs/CHANGES.txt
==============================================================================
--- MoreFieldsAndWidgets/archetypes.referencebrowserwidget/trunk/docs/CHANGES.txt	(original)
+++ MoreFieldsAndWidgets/archetypes.referencebrowserwidget/trunk/docs/CHANGES.txt	Sat May 28 00:50:40 2011
@@ -10,6 +10,10 @@
 * Fix undefined variable checkPermission 
   [kiorky]
 
+* Fix error in refbrowser_moveReferenceDown and refbrowser_moveReferenceUp
+  which caused page reloads when a referenced item was moved twice.
+  [cewing]
+
 
 2.4.1 - 2011-05-12
 ------------------

Modified: MoreFieldsAndWidgets/archetypes.referencebrowserwidget/trunk/src/archetypes/referencebrowserwidget/skins/referencebrowser/referencebrowser.js
==============================================================================
--- MoreFieldsAndWidgets/archetypes.referencebrowserwidget/trunk/src/archetypes/referencebrowserwidget/skins/referencebrowser/referencebrowser.js	(original)
+++ MoreFieldsAndWidgets/archetypes.referencebrowserwidget/trunk/src/archetypes/referencebrowserwidget/skins/referencebrowser/referencebrowser.js	Sat May 28 00:50:40 2011
@@ -280,10 +280,12 @@
     arrows = newelem.getElementsByTagName("a");
     arrows[0].onclick = function () {
         refbrowser_moveReferenceUp(this);
+        return false;
     };
     // down arrow
     arrows[1].onclick = function () {
         refbrowser_moveReferenceDown(this);
+        return false;
     };
 
     elem.parentNode.insertBefore(newelem, prevelem);
@@ -325,10 +327,12 @@
     arrows = newelem.getElementsByTagName("a");
     arrows[0].onclick = function () {
         refbrowser_moveReferenceUp(this);
+        return false;
     };
     // down img
     arrows[1].onclick = function () {
         refbrowser_moveReferenceDown(this);
+        return false;
     };
 
     nextelem = document.getElementById('ref-' + widget_id + '-' + (pos + 1));

------------------------------------------------------------------------------
vRanger cuts backup time in half-while increasing security.
With the market-leading solution for virtual backup and recovery, 
you get blazing-fast, flexible, and affordable data protection.
Download your free trial now. 
http://p.sf.net/sfu/quest-d2dcopy1
_______________________________________________
Archetypes-checkins mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/archetypes-checkins