Changeset 12034

Timestamp:

01/14/08 21:32:50 (12 months ago)

Author:

peller

Message:

Just use Boolean persist for now. Realized that Number or Date isn't possible in the parser, so the previous example was really coming through as expires: null, not expires: 1. expires: yyyy-mm-dd is funky but perhaps not really useful? Refs #5585

Location:

dijit/branches/newlayout

Files:

• layout/BorderContainer.js (modified) (4 diffs)
• tests/layout/test_BorderContainer.html (modified) (1 diff)

dijit/branches/newlayout/layout/BorderContainer.js

@@ -14 +14 @@
         //      Provides layout in 5 regions, a center and borders along its 4 sides.
         //
-        // details
+        // description
         //      A BorderContainer is a box with a specified size (like style="width: 500px; height: 500px;"),
-        //      that contains children widgets marked with "position" of "top", "bottom", "leading", "trailing", "center".
-        //  Children will be laid out inside the edges of the box with the remaining space left for the center.
-        //  Optional splitters may be specified on the edge widgets to make them resizable by the user.
+        //      that contains children widgets marked with "position" of "top", "bottom", "leading", "trailing",
+        // "left", "right", or "center".  Children along the edge will be laid out according to the width
+        //  or height dimensions specified in CSS, with the remaining space left for the center.
         //  The outer size must be specified on the BorderContainer node.  Width must be specified for the sides
         //  and height for the top and bottom.
         //  "left" and "right" may be used interchangably for "leading" and "trailing" except that those terms do
         //  not reflect the fact that they will be reversed in right-to-left environments.
+        //  Optional splitters may be specified on the edge widgets to make them resizable by the user.
         //
         // usage
@@ -39 +40 @@
         design: "headline",
 
+        // liveSplitters: Boolean
+        //  specifies whether splitters resize as you drag (true) or only upon mouseup (false)
         liveSplitters: true,
 
-        // persist: Number or Date
-        //              Save splitter positions in a cookie.  Persist is off (NaN) by default.
-        persist: new Date(""),  // NaN
+        // persist: Boolean
+        //              Save splitter positions in a cookie.
+        persist: false, // Boolean
 
         postCreate: function(){
@@ -320 +323 @@
 
                 this._cookieName = this.container.id + "_" + this.region;
-                if(!isNaN(this.container.persist)){
+                if(this.container.persist){
                         // restore old size
                         var persistSize = dojo.cookie(this._cookieName);
@@ -377 +380 @@
                 }
 
-                var expires = this.container.persist;
-                if(!isNaN(expires)){
-                        dojo.cookie(this._cookieName, this.child.domNode.style[this.horizontal ? "height" : "width"]/*this.domNode.style[this.region]*/, {expires: expires});
+                if(this.container.persist){
+                        dojo.cookie(this._cookieName, this.child.domNode.style[this.horizontal ? "height" : "width"]);
                 }
         },

dijit/branches/newlayout/tests/layout/test_BorderContainer.html

@@ -87 +87 @@
 <p>Sidebar layout, liveSplitters: false, persist: 1 day</p>
 
-<div dojoType="dijit.layout.BorderContainer" design="sidebar" liveSplitters="false" persist="1"
+<div dojoType="dijit.layout.BorderContainer" design="sidebar" liveSplitters="false" persist="true"
         style="border: 20px solid black; width: 90%; height: 300px; padding: 10px;"
 >