Changeset 265 for branches/1.1dev/lib

Timestamp:

Jul 3, 2007 3:35:31 AM (17 years ago)

Author:

quinn

Message:

Improved(?) getTextColor function

File:

• branches/1.1dev/lib/Utilities.inc.php (modified) (2 diffs)

branches/1.1dev/lib/Utilities.inc.php

@@ -173 +173 @@
     
 /**
- * Generates a hexadecibal html color based on provided word.
+ * Generates a hexadecimal html color based on provided word.
  *
  * @access public
@@ -183 +183 @@
 function getTextColor($text, $method=1)
 {
-    $r = substr(md5($text), 0, 1);
-    $g = substr(md5($text), 1, 1);
-    $b = substr(md5($text), 2, 1);
+    $hash = md5($text);
+    $rgb = array(
+        substr($hash, 0, 1),
+        substr($hash, 1, 1),
+        substr($hash, 2, 1),
+        substr($hash, 3, 1),
+        substr($hash, 4, 1),
+        substr($hash, 5, 1),
+    );
 
     switch ($method) {
+    case 1 :
+    default :
+        // Reduce all hex values slighly to avoid all white.
+        array_walk($rgb, create_function('&$v', '$v = dechex(round(hexdec($v) * 0.87));'));
+        break;
     case 2 :
-        if (hexdec($r) > hexdec('c')) {
-            $r = dechex(hexdec('f') - hexdec($r));
-        }
-        if (hexdec($g) > hexdec('c')) {
-            $g = dechex(hexdec('f') - hexdec($g));
-        }
-        if (hexdec($b) > hexdec('c')) {
-            $b = dechex(hexdec('f') - hexdec($b));
+        foreach ($rgb as $i => $v) {
+            if (hexdec($v) > hexdec('c')) {
+                $rgb[$i] = dechex(hexdec('f') - hexdec($v));
+            }
         }
         break;
-        
-    case 1 :
-    default :
-        $r = dechex(round(hexdec($r) * .8));
-        $g = dechex(round(hexdec($g) * .8));
-        $b = dechex(round(hexdec($b) * .6));
-        break;
-    }
-
-    return $r . $r . $g . $g . $b . $b;
+    }
+
+    return join('', $rgb);
 }