remove logging and floor values instead of 2hexing them

Author: mwcz

elements/pfe-avatar/src/hslrgb.js

@@ -39,38 +39,14 @@ export function hsl2rgb(_H, _S, _L) {
 
     a = 2 * L - b;
 
-    R = 255 * h2rgb(a, b, H + 1 / 3);
-    G = 255 * h2rgb(a, b, H);
-    B = 255 * h2rgb(a, b, H - 1 / 3);
+    R = Math.floor(255 * h2rgb(a, b, H + 1 / 3));
+    G = Math.floor(255 * h2rgb(a, b, H));
+    B = Math.floor(255 * h2rgb(a, b, H - 1 / 3));
   }
 
   return [R, G, B];
 }
 
-/**
- * Converts a decimal number into a hexadecimal number, left-padded to a given length.
- */
-function toHex(d, len) {
-  return ("0" + Math.round(d).toString(16)).slice(-len);
-}
-
-/**
- * Converts an HSL color to a hexadecimal color.
- */
-export function hsl2hex(_H, _S, _L) {
-  return rgb2hex(hsl2rgb(_H, _S, _L));
-}
-
-/**
- * Converts an RGB color to a hexadecimal color.
- */
-export function rgb2hex(_R, _G, _B) {
-  console.log(`converting ${_R}, ${_G}, ${_B} to hex`);
-  const ret = "#" + [_R, _G, _B].map(c => toHex(c, 2)).join("");
-  console.log(`done ${ret}`);
-  return ret;
-}
-
 /**
  * Convert an RGBcolor to HSL.
  *

elements/pfe-avatar/src/pfe-avatar.js

@@ -1,6 +1,6 @@
 import PFElement from "../pfelement/pfelement.js";
 import { hash } from "./djb-hash.js";
-import { hsl2rgb, rgb2hsl, rgb2hex } from "./hslrgb.js";
+import { hsl2rgb, rgb2hsl } from "./hslrgb.js";
 
 class PfeAvatar extends PFElement {
   static get tag() {
@@ -129,9 +129,7 @@ class PfeAvatar extends PFElement {
           );
           if (pattern) {
             pattern.shift();
-            console.log("pattern", pattern);
             const color = pattern.map(c => parseInt(c, 16));
-            console.log("color", color);
             this._registerColor(color);
           } else {
             this.log(`[pfe-avatar] invalid color ${colorCode}`);
@@ -144,16 +142,15 @@ class PfeAvatar extends PFElement {
 
   static _registerColor(color) {
     PfeAvatar.colors.push({
-      color1: rgb2hex(...color),
-      color2: rgb2hex(...this._adjustColor(color))
+      color1: `rgb(${color.join(",")})`,
+      color2: `rgb(${this._adjustColor(color).join(",")})`
     });
   }
 
   static _adjustColor(color) {
     const dark = 0.1;
     const l_adj = 0.1; // luminance adjustment
     const hsl = rgb2hsl(...color);
-    console.log(`${color} -> ${hsl}`);
 
     // if luminance is too dark already, then lighten the alternate color
     // instead of darkening it.