Changeset 6651

Timestamp:

03/28/08 01:38:08 (10 months ago)

Author:

euzuro

Message:

straighten things out once and for all in framed world regarding relative positioning. now that the underclasses (gofigure) are working as expected, we can too. the only time we need to bother ourselves is when there has been a moveTo(), not on a setSize() (since that function will automatically call moveTo()). now our calculateRelativePosition() is functioning correctly -- it doesn't update anything, it only just returns the relative position. the updating gets taken care of in moveTo(), just as it should.

Files:

• sandbox/euzuro/pop/lib/OpenLayers/Popup/Framed.js (modified) (5 diffs)

sandbox/euzuro/pop/lib/OpenLayers/Popup/Framed.js

@@ -94 +94 @@
-            //based on our decided relativePostion, set the current padding
-            // this keeps us from getting into trouble 
-this.relativePosition
+
-        }
-
@@ -151 +151 @@
-    },
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
-    },
-    
@@ -196 +195 @@
-            relativePosition = 
-                OpenLayers.Popup.Anchored.prototype.calculateRelativePosition.apply(this, arguments);
-
-            //update the frame's padding and offset for the new 
-            // relative position
-            this.updateRelativePosition(relativePosition);
-        }
-
@@ -207 +202 @@
-    /**
-     * Method: updateRelativePosition
-
-
-
-
-
-
+
+
+
-        //update the padding
-
+this.
-        
-        //update the position of our close box to new padding
@@ -227 +219 @@
-                                      this.padding.top + "px";
-        }        
+
+        this.updateFrames();
-    },
-