Revision: 3808
          http://vexi.svn.sourceforge.net/vexi/?rev=3808&view=rev
Author:   clrg
Date:     2010-02-26 13:20:47 +0000 (Fri, 26 Feb 2010)

Log Message:
-----------
Clean up

Modified Paths:
--------------
    trunk/widgets/org.vexi.widgets/src/org/vexi/lib/layout/aspect.t
    trunk/widgets/org.vexi.widgets/src/org/vexi/lib/role/popupmanager.t
    trunk/widgets/org.vexi.widgets/src_dev/visualtest/layout/aspect.t

Modified: trunk/widgets/org.vexi.widgets/src/org/vexi/lib/layout/aspect.t
===================================================================
--- trunk/widgets/org.vexi.widgets/src/org/vexi/lib/layout/aspect.t     
2010-02-26 11:35:35 UTC (rev 3807)
+++ trunk/widgets/org.vexi.widgets/src/org/vexi/lib/layout/aspect.t     
2010-02-26 13:20:47 UTC (rev 3808)
@@ -3,12 +3,6 @@
 <vexi xmlns:ui="vexi://ui" xmlns:meta="vexi://meta" xmlns:util="vexi.util">
     <meta:doc>
         <author>Charles Goodwin</author>
-        <todo>
-            * border-left/right/top/bottom
-            * take more than one argument to border like CSS
-            * depth read function when border is image based
-            * depth should be read-only with image borders
-        </todo>
     </meta:doc>
     
     <ui:box redirect=":$content">

Modified: trunk/widgets/org.vexi.widgets/src/org/vexi/lib/role/popupmanager.t
===================================================================
--- trunk/widgets/org.vexi.widgets/src/org/vexi/lib/role/popupmanager.t 
2010-02-26 11:35:35 UTC (rev 3807)
+++ trunk/widgets/org.vexi.widgets/src/org/vexi/lib/role/popupmanager.t 
2010-02-26 13:20:47 UTC (rev 3808)
@@ -71,11 +71,12 @@
             $container.display = popcount>0;
             // use default group if none is specified
             var g = p.popgroup ? p.popgroup : "default";
-            //vexi.trace("popcount--: "+popcount+" ("+g+")");
-            if (popforegs[g])
-               popforegs[g].display = false;
-            if (popgroups[g]) 
-                   popgroups[g] = null;
+            if (popforegs[g]) {
+                popforegs[g].display = false;
+            }
+            if (popgroups[g]) {
+                popgroups[g] = null;
+            }
         }
         
         /** private function to pop up box 'v' */
@@ -112,7 +113,6 @@
             // handle container display
             if (p.display) {
                 popcount++;
-                //vexi.trace("popcount++: "+popcount+" ("+g+")");
                 $container.display = popcount>0;
                 if (p.visible) {
                     p.surface_x ++= syncXPos;
@@ -175,7 +175,7 @@
             
             if (popforegs[g]) {
                 // if group exists slot into it
-                popforegs[g][popforegs[g].numchildren] = p;
+                popforegs[g].add(p);
             
             } else {
                 // initialize new group, especially its foreground box
@@ -198,13 +198,6 @@
                     }
                 }
                 $container[n] = fg;
-
-                // simple cleanup on frame close (slowdown Vexi leaks)
-//                surface.frame.Close ++= function(v) {
-//                    cascade = v;
-//                    popforegs[g] = null;
-//                    $container[n] = null;
-//                }
             }
         }
 
@@ -248,9 +241,9 @@
         surface.frame.Focused ++= function(v) {
             cascade = v;
             if (!v) {
-                for (var key in popgroups) {
-                    if (popgroups[key] and !popgroups[key].ignorefocusloss) {
-                        popgroups[key].v_popmaster.popdown = true;
+                for (var key,grp in popgroups) {
+                    if (grp and !grp.ignorefocusloss) {
+                        grp.v_popmaster.popdown = true;
                     }
                 }
             }

Modified: trunk/widgets/org.vexi.widgets/src_dev/visualtest/layout/aspect.t
===================================================================
--- trunk/widgets/org.vexi.widgets/src_dev/visualtest/layout/aspect.t   
2010-02-26 11:35:35 UTC (rev 3807)
+++ trunk/widgets/org.vexi.widgets/src_dev/visualtest/layout/aspect.t   
2010-02-26 13:20:47 UTC (rev 3808)
@@ -20,4 +20,4 @@
         vexi.ui.frame = thisbox;
         
     </ui:box>
-</vexi>
\ No newline at end of file
+</vexi>


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

------------------------------------------------------------------------------
Download Intel&#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Vexi-svn mailing list
Vexi-svn@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/vexi-svn

Reply via email to