Flex fail

diff --git a/TODO.txt b/TODO.txt
index b998935641ca1897cdcd6c3084e1ad0e492100e4..542fe5ced542d0e49a11a6bd1c55852d92e63bdd 100644 (file)
--- a/TODO.txt
+++ b/TODO.txt
@@ -61,7 +61,6 @@ Potlatch 2: main outstanding issues
 == Rendering (Halcyon) ==
 
 * StyleList and everything that iterates over it needs to cope with arbitrary sublayers
-* Occasionally way selected highlight appears in black, not yellow
 * halcyon_viewer needs updating for new tileurl stuff
 * { interactive: no; } - stop items being clicked on (and their maxwidth being taken into account)
 * Shields

diff --git a/net/systemeD/halcyon/styleparser/RuleSet.as b/net/systemeD/halcyon/styleparser/RuleSet.as
index 2ab00b79839a46a7ff539976414f7253bd3e5f2f..4a0738198d192d839cab22639a80a9aa4a33a36a 100644 (file)
--- a/net/systemeD/halcyon/styleparser/RuleSet.as
+++ b/net/systemeD/halcyon/styleparser/RuleSet.as
@@ -427,7 +427,7 @@ package net.systemeD.halcyon.styleparser {
                        var styles:Array=[];
                        var t:Object=new Object();
                        var o:Object=new Object();
-                       var a:String, k:String;
+                       var a:String, k:String, v:*;
 
                        // Create styles\10
                        var ss:ShapeStyle =new ShapeStyle() ;
@@ -468,15 +468,14 @@ package net.systemeD.halcyon.styleparser {
                        for (a in t) {
                                // Parse properties
                                // ** also do units, e.g. px/pt
-                               if (a.match(COLOR)) {
-                                       t[a] = parseCSSColor(t[a]);
-                               }
+                               if (a.match(COLOR)) { v = parseCSSColor(t[a]); }
+                                              else { v = t[a]; }
                                
                                // Set in styles
-                               if      (ss.hasOwnProperty(a)) { ss.setPropertyFromString(a,t[a]); }
-                               else if (ps.hasOwnProperty(a)) { ps.setPropertyFromString(a,t[a]); }
-                               else if (ts.hasOwnProperty(a)) { ts.setPropertyFromString(a,t[a]); }
-                               else if (hs.hasOwnProperty(a)) { hs.setPropertyFromString(a,t[a]); }
+                               if      (ss.hasOwnProperty(a)) { ss.setPropertyFromString(a,v); }
+                               else if (ps.hasOwnProperty(a)) { ps.setPropertyFromString(a,v); }
+                               else if (ts.hasOwnProperty(a)) { ts.setPropertyFromString(a,v); }
+                               else if (hs.hasOwnProperty(a)) { hs.setPropertyFromString(a,v); }
                        }
 
                        // Add each style to list
@@ -515,12 +514,11 @@ package net.systemeD.halcyon.styleparser {
 
         public static function parseCSSColor(colorStr:String):uint {
             colorStr = colorStr.toLowerCase();
-            if (CSSCOLORS[colorStr])
+            if (CSSCOLORS[colorStr]) {
                 return CSSCOLORS[colorStr];
-            else {
+            } else {
                 var match:Object = HEX.exec(colorStr);
-                if ( match )
-                    return Number("0x"+match[1]);
+                if ( match ) { return Number("0x"+match[1]); }
             }
             return 0;
         }