Revision: 2795
          http://vexi.svn.sourceforge.net/vexi/?rev=2795&view=rev
Author:   clrg
Date:     2008-02-24 16:02:52 -0800 (Sun, 24 Feb 2008)

Log Message:
-----------
Fix (and improve method) of selection effect for cells

Modified Paths:
--------------
    trunk/widgets/org.vexi.widgets/src/org/vexi/theme/classic/cell.t
    trunk/widgets/org.vexi.widgets/src/org/vexi/theme/classic/row.t

Modified: trunk/widgets/org.vexi.widgets/src/org/vexi/theme/classic/cell.t
===================================================================
--- trunk/widgets/org.vexi.widgets/src/org/vexi/theme/classic/cell.t    
2008-02-24 21:00:59 UTC (rev 2794)
+++ trunk/widgets/org.vexi.widgets/src/org/vexi/theme/classic/cell.t    
2008-02-25 00:02:52 UTC (rev 2795)
@@ -8,8 +8,17 @@
     
     <cell padding="2">
         thisbox.datatype ++= static.datatypeWrite;
+        thisbox.selected ++= static.selectedWrite;
     </cell>
     
+    /** apply suitable effects when selected */
+    static.selectedWrite = function(v) {
+        cascade = v;
+        if (trapee.datatype == "toggle")
+            trapee[0].fill = v?theme.image.check_highlighted:theme.image.check;
+        else trapee.textcolor = 
v?theme.settings..textcolor_selected:theme.settings..textcolor;
+    }
+    
     /** return value as substitute for text */
     static.toggleTextRead = function() { return trapee.value; }
     
@@ -19,12 +28,6 @@
         return;
     }
     
-    /** display nice white check when selected */
-    static.toggleTextcolorWrite = function(v) {
-        cascade = v;
-        trapee[0].fill = theme.image["check" + (v == "white" ? "_highlighted" 
: "")];
-    }
-    
     /** sets up theme specific datatype effects */
     static.datatypeWrite = function(v) {
         // apply toggle effect / traps
@@ -37,13 +40,11 @@
             trapee[0].shrink = true;
             trapee.text ++= static.toggleTextRead;
             trapee.text ++= static.toggleTextWrite;
-            trapee.textcolor ++= static.toggleTextcolorWrite;
         // remove toggle effect / traps
         } else if (trapee.datatype == "toggle") {
             trapee[0] = null;
             trapee.text --= static.toggleTextRead;
             trapee.text --= static.toggleTextWrite;
-            trapee.textcolor --= static.toggleTextcolorWrite;
         }
         cascade = v;
     }

Modified: trunk/widgets/org.vexi.widgets/src/org/vexi/theme/classic/row.t
===================================================================
--- trunk/widgets/org.vexi.widgets/src/org/vexi/theme/classic/row.t     
2008-02-24 21:00:59 UTC (rev 2794)
+++ trunk/widgets/org.vexi.widgets/src/org/vexi/theme/classic/row.t     
2008-02-25 00:02:52 UTC (rev 2795)
@@ -14,16 +14,12 @@
         }
         
         thisbox.selected ++= function(v) {
-            if (v) {
-                fill = focused ? .settings..fill_selected : 
.settings..fill_selected_unfocused;
-                for (var i=0; numchildren>i; i++)
-                    thisbox[i].textcolor = .settings..textcolor_selected;
-            } else {
-                fill = rowfill;
-                for (var i=0; numchildren>i; i++)
-                    thisbox[i].textcolor = .settings..textcolor;
-            }
             cascade = v;
+            fill = v ? focused ? .settings..fill_selected
+                               : .settings..fill_selected_unfocused
+                     : rowfill;
+            for (var i=0; numchildren>i; i++)
+                thisbox[i].selected = v;
         }
         
     </lib:row>


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

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Vexi-svn mailing list
Vexi-svn@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/vexi-svn

Reply via email to