From 5d14efb20159441d40306ba64e5f38b1e1d64de5 Mon Sep 17 00:00:00 2001
From: cmack <conoramck@gmail.com>
Date: Tue, 28 Jul 2020 14:19:46 +0100
Subject: [PATCH] bar chart

---
 .../ComponentPropertiesPanel.svelte           |   1 +
 .../standard-components/src/Chart/Bar.svelte  | 130 ++++++++++++++----
 2 files changed, 105 insertions(+), 26 deletions(-)

diff --git a/packages/builder/src/components/userInterface/ComponentPropertiesPanel.svelte b/packages/builder/src/components/userInterface/ComponentPropertiesPanel.svelte
index 38e4dd67a9..fcdd22db4f 100644
--- a/packages/builder/src/components/userInterface/ComponentPropertiesPanel.svelte
+++ b/packages/builder/src/components/userInterface/ComponentPropertiesPanel.svelte
@@ -145,6 +145,7 @@
     margin-top: 10px;
     flex: 1 1 auto;
     min-height: 0;
+    overflow-y: auto;
   }
 
   .instance-name {
diff --git a/packages/standard-components/src/Chart/Bar.svelte b/packages/standard-components/src/Chart/Bar.svelte
index ef16a7f66c..2bea43cb14 100644
--- a/packages/standard-components/src/Chart/Bar.svelte
+++ b/packages/standard-components/src/Chart/Bar.svelte
@@ -1,14 +1,25 @@
 <script>
-  import Chart, { getColorSchema, getChartGradient } from "./Chart.svelte"
+  import { getColorSchema, getChartGradient } from "./Chart.svelte"
+  import britecharts from "britecharts"
+  import { onMount } from "svelte"
 
+  import { select } from "d3-selection"
+  import shortid from "shortid"
   /*
     ISSUES:
     nameLabel, valueLabel - what are these? Seem to break whenever passed a string or number. What type?
     https://github.com/britecharts/britecharts/blob/a2c45ab023112b36e14f47c278e3a1e1c05f8383/src/charts/bar.js#L145
   */
 
-  let type = "bar"
   let tooltip
+  const _id = shortid.generate()
+  const chart = britecharts.bar()
+  const chartClass = `bar-container-${_id}`
+  const legendClass = `legend-container-${_id}`
+
+  let chartElement = null
+  let chartContainer = null
+  let tooltipContainer = null
 
   export let customMouseOver = () => tooltip.show()
   export let customMouseMove = (datapoint, colorMapping, x, y) =>
@@ -37,31 +48,98 @@
 
   export let useLegend = true
 
+  onMount(() => {
+    //call charts props in here
+    if (chartElement) {
+      chartContainer = select(`.${chartClass}`)
+      bindChartUIProps()
+      bindChartEvents()
+      bindChartTooltip()
+      chartContainer.datum(data).call(chart)
+    }
+  })
+
+  function bindChartUIProps() {
+    if (color) {
+      chart.colorSchema(colorSchema)
+    }
+    if (gradient) {
+      chart.chartGradient(chartGradient)
+    }
+    if (xAxisLabel) {
+      chart.xAxisLabel(xAxisLabel)
+    }
+    if (yAxisLabel) {
+      chart.yAxisLabel(yAxisLabel)
+    }
+    if (betweenBarsPadding) {
+      chart.betweenBarsPadding(betweenBarsPadding)
+    }
+    if (enableLabels) {
+      chart.enableLabels(enableLabels)
+    }
+    if (hasSingleBarHighlight) {
+      chart.hasSingleBarHighlight(hasSingleBarHighlight)
+    }
+    if (height) {
+      chart.height(height)
+    }
+    if (width) {
+      chart.width(width)
+    }
+    if (isAnimated) {
+      chart.isAnimated(isAnimated)
+    }
+    if (isHorizontal) {
+      chart.isHorizontal(isHorizontal)
+    }
+    if (labelOffset) {
+      chart.labelOffset(labelOffset)
+    }
+    if (labelsNumberFormat) {
+      chart.labelsNumberFormat(labelsNumberFormat)
+    }
+    if (labelSize) {
+      chart.labelSize(labelSize)
+    }
+    if (locale) {
+      chart.locale(locale)
+    }
+    if (nameLabel) {
+      chart.nameLabel(nameLabel)
+    }
+    if (numberFormat) {
+      chart.numberFormat(numberFormat)
+    }
+  }
+
+  function bindChartEvents() {
+    if (customMouseMove) {
+      chart.on("customMouseMove", customMouseMove)
+    }
+    if (customMouseOut) {
+      chart.on("customMouseOut", customMouseOut)
+    }
+    if (customMouseOver) {
+      chart.on("customMouseOver", customMouseOver)
+    }
+    if (customClick) {
+      chart.on("customClick", customClick)
+    }
+  }
+
+  function bindChartTooltip() {
+    tooltip = britecharts.miniTooltip()
+    debugger
+    tooltipContainer = select(`.${chartClass} .metadata-group`)
+    tooltipContainer.datum([]).call(tooltip)
+  }
+
   $: colorSchema = getColorSchema(color)
   $: chartGradient = getChartGradient(gradient)
 </script>
 
-<Chart
-  bind:tooltip
-  {type}
-  {data}
-  on={{ customClick, customMouseMove, customMouseOut, customMouseOver }}
-  useTooltip
-  {useLegend}
-  {betweenBarsPadding}
-  {chartGradient}
-  {colorSchema}
-  {enableLabels}
-  {hasSingleBarHighlight}
-  {height}
-  {width}
-  {isAnimated}
-  {isHorizontal}
-  {labelOffset}
-  {labelsNumberFormat}
-  {labelSize}
-  {locale}
-  {nameLabel}
-  {numberFormat}
-  {xAxisLabel}
-  {yAxisLabel} />
+<div bind:this={chartElement} class={chartClass} />
+{#if useLegend}
+  <div class={legendClass} />
+{/if}