Changeset 2157

Timestamp:

09/08/10 16:30:57 (17 months ago)

Author:

johnnyoffline

Message:

Fixed a scoping issue with the move arrows and their parent containers.

Files:

• plugins/apostrophePlugin/trunk/web/js/a.js (modified) (4 diffs)

plugins/apostrophePlugin/trunk/web/js/a.js

@@ -495 +495 @@
                         // TODO: with a little more finesse we could support saving it with
                         // a rank, but think about how messy that might get
-                  slots.find('.a-arrow-up,.a-arrow-down').hide();
+                  slots.find('.a-slot-controls .a-move').hide();
                         return;
                 }
@@ -529 +529 @@
         function slotUpdateMoveButtons(id, name, slot, n, slots, updateAction)
         {
+                var up = $(slot).find('.a-arrow-up');
+                var down = $(slot).find('.a-arrow-down');
+                                        
                 if (n > 0)
                 {
                         // TODO: this is not sensitive enough to nested areas
-                        $(slot).find('.a-arrow-up').show().unbind('click').click(function() {
+                        up.parent().show();
+                        up.unbind('click').click(function() {
                                 // It would be nice to confirm success here in some way
                                 $.get(updateAction, { id: id, name: name, permid: $(slot).data('a-permid'), up: 1 });
@@ -542 +546 @@
                 else
                 {
-                  $(slot).find('.a-arrow-up').hide();
+                  up.parent().hide();
                 }
                 if (n < (slots.length - 1))
                 {
-                        $(slot).find('.a-arrow-down').show().unbind('click').click(function() {
+
+                        down.parent().show();
+                        down.unbind('click').click(function() {
                                 // It would be nice to confirm success here in some way
                                 $.get(updateAction, { id: id, name: name, permid: $(slot).data('a-permid'), up: 0 });
@@ -556 +562 @@
                 else
                 {
-                        $(slot).find('.a-arrow-down').hide();
-                }
-        }
+                        down.parent().hide();
+                }
+        }sv
         
         function slotShowEditViewPreloaded(pageid, name, permid)