From 77aa35be7d5c2455a6d94c2a956944b4732eb48f Mon Sep 17 00:00:00 2001
From: Ivan Mikheykin <ivan.mikheykin@flant.com>
Date: Mon, 3 Sep 2018 16:33:58 +0300
Subject: [PATCH] set discrete color legend width as graph width

---
 src/color_legend.js         | 47 ++++++++-----------------------------
 src/css/status-heatmap.scss |  4 ++--
 src/options_editor.html     |  2 +-
 3 files changed, 13 insertions(+), 40 deletions(-)

diff --git a/src/color_legend.js b/src/color_legend.js
index 49d3296..9cdc2dc 100644
--- a/src/color_legend.js
+++ b/src/color_legend.js
@@ -38,10 +38,6 @@ mod.directive('optionsColorLegend', function() {
         } else if (panel.color.mode === 'opacity') {
           let colorOptions = panel.color;
           drawSimpleOpacityLegend(elem, colorOptions);
-        } else if (panel.color.mode === 'discrete') {
-          let colorOptions = panel.color;
-          let colorScale = getDiscreteColorScale(colorOptions, legendWidth);
-          drawSimpleDiscreteColorLegend(elem, colorOptions, colorScale);
         }
       }
     }
@@ -159,9 +155,17 @@ function drawDiscreteColorLegend(elem, colorOptions, rangeFrom, rangeTo, maxValu
   let legend = d3.select(legendElem.get(0));
   clearLegend(elem);
 
+  // FIXME: hack for long discrete tooltips. Spectrum will use 168px after editing.
+  // FIXME: render all tooltips as svg, get max width. calc min between tooltip width and graphWidth/tooltip count.
+  // (min+2 for padding)*count is a new legend width.
+  // TODO add clippath or mask if text overlapped!
+  let $heatmap = $(elem).parent().parent().parent().find('.status-heatmap-panel');
+  let graphWidth  = $heatmap.find('svg').attr("width");
+  legendElem.attr("width", graphWidth);
+
   let thresholds = colorOptions.thresholds;
 
-  let legendWidth = Math.floor(legendElem.outerWidth()) - 30;
+  let legendWidth = Math.floor(graphWidth) - 30;   // Math.floor(legendElem.outerWidth()) - 30;
   let legendHeight = legendElem.attr("height");
 
   let valuesNumber = thresholds.length;
@@ -297,6 +301,7 @@ function drawSimpleColorLegend(elem, colorScale) {
 
 function drawSimpleOpacityLegend(elem, options) {
   let legendElem = $(elem).find('svg');
+  let graphElem = $(elem);
   clearLegend(elem);
 
   let legend = d3.select(legendElem.get(0));
@@ -330,32 +335,6 @@ function drawSimpleOpacityLegend(elem, options) {
   }
 }
 
-function drawSimpleDiscreteColorLegend(elem, colorOptions, colorScale) {
-  let thresholds = colorOptions.thresholds;
-
-  let legendElem = $(elem).find('svg');
-  clearLegend(elem);
-
-  let legendWidth = Math.floor(legendElem.outerWidth());
-  let legendHeight = legendElem.attr("height");
-
-  if (legendWidth) {
-    let valuesNumber = thresholds.length;
-    let rangeStep  = Math.floor(legendWidth / valuesNumber);
-    let valuesRange = d3.range(0, legendWidth, rangeStep);
-
-    let legend = d3.select(legendElem.get(0));
-    var legendRects = legend.selectAll(".status-heatmap-discrete-legend-rect").data(valuesRange);
-
-    legendRects.enter().append("rect")
-      .attr("x", d => d)
-      .attr("y", 0)
-      .attr("width", rangeStep + 1) // Overlap rectangles to prevent gaps
-      .attr("height", legendHeight)
-      .attr("stroke-width", 0)
-      .attr("fill", d => colorScale(d));
-  }
-}
 
 function clearLegend(elem) {
   let legendElem = $(elem).find('svg');
@@ -393,12 +372,6 @@ function getDiscreteColorScale(colorOptions, maxValue, minValue = 0) {
       return color
     }
     return 'rgba(0,0,0,0)';
-    // for (let i = 0; i < thresholdValues.length; i++ ) {
-    //   if (d == thresholdValues[i]) {
-    //     return thresholdColors[i];
-    //   }
-    // }
-    // return thresholdColors[0];
   };
 
   let inputRangeScaler = d3.scaleLinear().domain([start, end]).range([0, thresholdColors.length+1]);
diff --git a/src/css/status-heatmap.scss b/src/css/status-heatmap.scss
index 575248f..35fa111 100644
--- a/src/css/status-heatmap.scss
+++ b/src/css/status-heatmap.scss
@@ -67,8 +67,8 @@ $font-size-sm: 12px !default;
   //padding-top: 10px;
 
   svg {
-    width: 100%;
-    max-width: 300px;
+    //width: 100%;
+    //max-width: 1300px;
     height: 24px;
     float: left;
     white-space: nowrap;
diff --git a/src/options_editor.html b/src/options_editor.html
index 27a34ef..d138fa3 100644
--- a/src/options_editor.html
+++ b/src/options_editor.html
@@ -91,7 +91,7 @@
         </div>
     </div>
 
-    <div class="gf-form" ng-show="ctrl.panel.color.mode !== 'discrete'">
+    <div class="gf-form">
       <options-color-legend></options-color-legend>
     </div>
   </div>