Bug 1335451 - Tweak thumbnail placeholder colors and make placeholder letters uppercase. r?gijs draft
authorDão Gottwald <dao@mozilla.com>
Wed, 01 Feb 2017 19:35:31 +0100
changeset 469139 faef07b48b1b1d469383097fbac82524632353b7
parent 468498 ee975d32deb9eaa5641f45428cd6a4b5b555a8f5
child 544101 01daa7ff2ef05f66443e191ee7e2f0ee3d9f11c9
push id43623
push userdgottwald@mozilla.com
push dateWed, 01 Feb 2017 18:36:38 +0000
reviewersgijs
bugs1335451
milestone54.0a1
Bug 1335451 - Tweak thumbnail placeholder colors and make placeholder letters uppercase. r?gijs MozReview-Commit-ID: 1s2MMPNBA5S
browser/base/content/newtab/sites.js
--- a/browser/base/content/newtab/sites.js
+++ b/browser/base/content/newtab/sites.js
@@ -253,23 +253,24 @@ Site.prototype = {
     }
     let uri = link.imageURI || PageThumbs.getThumbnailURL(this.url);
     thumbnail.style.backgroundImage = 'url("' + uri + '")';
 
     if (THUMBNAIL_PLACEHOLDER_ENABLED &&
         link.type == "history" &&
         link.baseDomain) {
       let placeholder = this._querySelector(".newtab-thumbnail.placeholder");
-      let hue = 0;
+      let charCodeSum = 0;
       for (let c of link.baseDomain) {
-        hue += c.charCodeAt(0);
+        charCodeSum += c.charCodeAt(0);
       }
-      hue %= 256;
-      placeholder.style.backgroundColor = "hsl(" + hue + ",50%,60%)";
-      placeholder.textContent = link.baseDomain.substr(0,1);
+      const COLORS = 16;
+      let hue = Math.round((charCodeSum % COLORS) / COLORS * 360);
+      placeholder.style.backgroundColor = "hsl(" + hue + ",80%,40%)";
+      placeholder.textContent = link.baseDomain.substr(0,1).toUpperCase();
     }
 
     if (link.enhancedImageURI) {
       let enhanced = this._querySelector(".enhanced-content");
       enhanced.style.backgroundImage = 'url("' + link.enhancedImageURI + '")';
 
       if (this.link.type != link.type) {
         this.node.setAttribute("type", "enhanced");