From bb5da598b7db6ea1c0ca1649f6b34bb1a4db3e07 Mon Sep 17 00:00:00 2001
From: Alex P <thealexpaxton@gmail.com>
Date: Thu, 12 Jul 2018 13:25:18 -0700
Subject: [PATCH] Remove zoom and xbrush from histogram

---
 ui/src/shared/components/HistogramChart.tsx | 27 ---------------------
 ui/src/types/histogram.ts                   |  5 ----
 2 files changed, 32 deletions(-)

diff --git a/ui/src/shared/components/HistogramChart.tsx b/ui/src/shared/components/HistogramChart.tsx
index c7c01208a..22f49459a 100644
--- a/ui/src/shared/components/HistogramChart.tsx
+++ b/ui/src/shared/components/HistogramChart.tsx
@@ -6,12 +6,10 @@ import HistogramChartAxes from 'src/shared/components/HistogramChartAxes'
 import HistogramChartBars from 'src/shared/components/HistogramChartBars'
 import HistogramChartTooltip from 'src/shared/components/HistogramChartTooltip'
 import HistogramChartSkeleton from 'src/shared/components/HistogramChartSkeleton'
-import XBrush from 'src/shared/components/XBrush'
 
 import extentBy from 'src/utils/extentBy'
 
 import {
-  TimePeriod,
   HistogramData,
   Margins,
   HoverData,
@@ -33,7 +31,6 @@ interface Props {
   height: number
   colors: HistogramColor[]
   colorScale: ColorScale
-  onZoom?: (TimePeriod) => void
   onBarClick?: (time: string) => void
 }
 
@@ -86,7 +83,6 @@ class HistogramChart extends PureComponent<Props, State> {
               yScale={yScale}
             />
           </g>
-          {this.xBrush}
           <g
             transform={bodyTransform}
             className="histogram-chart--bars"
@@ -117,24 +113,6 @@ class HistogramChart extends PureComponent<Props, State> {
     )
   }
 
-  private get xBrush(): JSX.Element {
-    const {onZoom} = this.props
-    const {xScale, adjustedWidth, adjustedHeight, bodyTransform} = this
-
-    if (onZoom) {
-      return (
-        <g className="histogram-chart--brush" transform={bodyTransform}>
-          <XBrush
-            xScale={xScale}
-            width={adjustedWidth}
-            height={adjustedHeight}
-            onBrush={this.handleBrush}
-          />
-        </g>
-      )
-    }
-  }
-
   private get xScale(): ScaleTime<number, number> {
     const {adjustedWidth} = this
     const {data} = this.props
@@ -196,11 +174,6 @@ class HistogramChart extends PureComponent<Props, State> {
     return Math.max(...counts)
   }
 
-  private handleBrush = (t: TimePeriod): void => {
-    this.props.onZoom(t)
-    this.setState({hoverData: null})
-  }
-
   private handleHover = (hoverData: HoverData): void => {
     this.setState({hoverData})
   }
diff --git a/ui/src/types/histogram.ts b/ui/src/types/histogram.ts
index 3c151da6d..1fb119785 100644
--- a/ui/src/types/histogram.ts
+++ b/ui/src/types/histogram.ts
@@ -7,11 +7,6 @@ export interface HistogramDatum {
   group: string
 }
 
-export interface TimePeriod {
-  start: UnixTime
-  end: UnixTime
-}
-
 export type HistogramData = HistogramDatum[]
 
 export type TooltipAnchor = 'left' | 'right'