Move toColor into channels

It operates on rgba channels, not a packed 32 bit int

src/common/Color.test.ts

@@ -29,6 +29,25 @@ describe('Color', () => {
         assert.equal(channels.toCss(0xf0, 0xf0, 0xf0), '#f0f0f0');
         assert.equal(channels.toCss(0xff, 0xff, 0xff), '#ffffff');
       });
+      it('should convert an rgba array to css hex string', () => {
+        assert.equal(channels.toCss(0x00, 0x00, 0x00, 0x00), '0x00000000');
+        assert.equal(channels.toCss(0x10, 0x10, 0x10, 0x10), '0x10101010');
+        assert.equal(channels.toCss(0x20, 0x20, 0x20, 0x20), '0x20202020');
+        assert.equal(channels.toCss(0x30, 0x30, 0x30, 0x30), '0x30303030');
+        assert.equal(channels.toCss(0x40, 0x40, 0x40, 0x40), '0x40404040');
+        assert.equal(channels.toCss(0x50, 0x50, 0x50, 0x50), '0x50505050');
+        assert.equal(channels.toCss(0x60, 0x60, 0x60, 0x60), '0x60606060');
+        assert.equal(channels.toCss(0x70, 0x70, 0x70, 0x70), '0x70707070');
+        assert.equal(channels.toCss(0x80, 0x80, 0x80, 0x80), '0x80808080');
+        assert.equal(channels.toCss(0x90, 0x90, 0x90, 0x90), '0x90909090');
+        assert.equal(channels.toCss(0xa0, 0xa0, 0xa0, 0xa0), '0xa0a0a0a0');
+        assert.equal(channels.toCss(0xb0, 0xb0, 0xb0, 0xb0), '0xb0b0b0b0');
+        assert.equal(channels.toCss(0xc0, 0xc0, 0xc0, 0xc0), '0xc0c0c0c0');
+        assert.equal(channels.toCss(0xd0, 0xd0, 0xd0, 0xd0), '0xd0d0d0d0');
+        assert.equal(channels.toCss(0xe0, 0xe0, 0xe0, 0xe0), '0xe0e0e0e0');
+        assert.equal(channels.toCss(0xf0, 0xf0, 0xf0, 0xf0), '0xf0f0f0f0');
+        assert.equal(channels.toCss(0xff, 0xff, 0xff, 0xff), '0xffffffff');
+      });
     });
 
     describe('toRgba', () => {

src/common/Color.ts

@@ -33,6 +33,13 @@ export namespace channels {
     // >>> 0 forces an unsigned int
     return (r << 24 | g << 16 | b << 8 | a) >>> 0;
   }
+
+  export function toColor(r: number, g: number, b: number, a?: number): IColor {
+    return {
+      css: channels.toCss(r, g, b, a),
+      rgba: channels.toRgba(r, g, b, a)
+    };
+  }
 }
 
 /**
@@ -70,7 +77,7 @@ export namespace color {
     if (!result) {
       return undefined;
     }
-    return rgba.toColor(
+    return channels.toColor(
       (result >> 24 & 0xFF),
       (result >> 16 & 0xFF),
       (result >> 8  & 0xFF)
@@ -142,14 +149,14 @@ export namespace css {
           $r = parseInt(css.slice(1, 2).repeat(2), 16);
           $g = parseInt(css.slice(2, 3).repeat(2), 16);
           $b = parseInt(css.slice(3, 4).repeat(2), 16);
-          return rgba.toColor($r, $g, $b);
+          return channels.toColor($r, $g, $b);
         }
         case 5: { // #rgba
           $r = parseInt(css.slice(1, 2).repeat(2), 16);
           $g = parseInt(css.slice(2, 3).repeat(2), 16);
           $b = parseInt(css.slice(3, 4).repeat(2), 16);
           $a = parseInt(css.slice(4, 5).repeat(2), 16);
-          return rgba.toColor($r, $g, $b, $a);
+          return channels.toColor($r, $g, $b, $a);
         }
         case 7: // #rrggbb
           return {
@@ -171,7 +178,7 @@ export namespace css {
       $g = parseInt(rgbaMatch[2]);
       $b = parseInt(rgbaMatch[3]);
       $a = Math.round((rgbaMatch[5] === undefined ? 1 : parseFloat(rgbaMatch[5])) * 0xFF);
-      return rgba.toColor($r, $g, $b, $a);
+      return channels.toColor($r, $g, $b, $a);
     }
 
     // Validate the context is available for canvas-based color parsing
@@ -342,17 +349,9 @@ export namespace rgba {
     return (fgR << 24 | fgG << 16 | fgB << 8 | 0xFF) >>> 0;
   }
 
-  // FIXME: Move this to channels NS?
   export function toChannels(value: number): [number, number, number, number] {
     return [(value >> 24) & 0xFF, (value >> 16) & 0xFF, (value >> 8) & 0xFF, value & 0xFF];
   }
-
-  export function toColor(r: number, g: number, b: number, a?: number): IColor {
-    return {
-      css: channels.toCss(r, g, b, a),
-      rgba: channels.toRgba(r, g, b, a)
-    };
-  }
 }
 
 export function toPaddedHex(c: number): string {

src/common/Types.d.ts

@@ -110,8 +110,8 @@ export interface ICharset {
 export type CharData = [number, string, number, number];
 
 export interface IColor {
-  css: string;
-  rgba: number; // 32-bit int with rgba in each byte
+  readonly css: string;
+  readonly rgba: number; // 32-bit int with rgba in each byte
 }
 export type IColorRGB = [number, number, number];