Changeset 12498

Timestamp:

02/18/08 13:33:51 (11 months ago)

Author:

peller

Message:

resize only those children impacted by a relayout. move another style attribute to CSS. Refs #4117 !strict

Location:

dijit/trunk

Files:

• layout/BorderContainer.js (modified) (2 diffs)
• themes/dijit.css (modified) (1 diff)

dijit/trunk/layout/BorderContainer.js

@@ -65 +65 @@
                 if(region){
                         dojo.addClass(child.domNode, "dijitBorderContainerPane");
-                        child.domNode.style.position = "absolute"; //FIXME: for some reason, setting this in CSS isn't good enough?
 
                         var ltr = dojo._isBodyLtr();
@@ -272 +271 @@
 don't do that.  See #3399, #2678, #3624 and #2955, #1988
 */
+
+                var resizeList = {};
+                if(changedRegion){
+                        resizeList[changedRegion] = resizeList.center = true;
+                        if(/top|bottom/.test(changedRegion) && this.design != "sidebar"){
+                                resizeList.left = resizeList.right = true;
+                        }else if(/left|right/.test(changedRegion) && this.design == "sidebar"){
+                                resizeList.top = resizeList.bottom = true;
+                        }
+                }
+
                 dojo.forEach(this.getChildren(), function(child){
-//PERF: don't have to resize children if(changedRegion)
-                        if(child.resize){
+                        if(child.resize && (!changedRegion || child.region in resizeList)){
 //                              console.log(this.id, ": resizing child id=" + child.id + " (region=" + child.region + "), style before resize is " +
 //                                                                       "{ t: " + child.domNode.style.top +

dijit/trunk/themes/dijit.css

@@ -634 +634 @@
 }
 
+.dijitBorderContainerPane {
+        position:absolute;
+}
+
 /* SplitContainer