Merge pull request #3156 from influxdata/polish/legend-hiding

Fix legend sticking around when opening CEO
pull/10616/head
Andrew Watkins 2018-04-09 16:03:42 -07:00 committed by GitHub
commit 0cf31351e5
1 changed files with 4 additions and 13 deletions

View File

@ -236,6 +236,7 @@ class Dygraph extends Component {
const newTime = this.eventToTimestamp(e) const newTime = this.eventToTimestamp(e)
this.props.onSetHoverTime(newTime) this.props.onSetHoverTime(newTime)
} }
this.setState({isHoveringThisGraph: true}) this.setState({isHoveringThisGraph: true})
} }
@ -243,23 +244,13 @@ class Dygraph extends Component {
if (this.props.onSetHoverTime) { if (this.props.onSetHoverTime) {
this.props.onSetHoverTime(NULL_HOVER_TIME) this.props.onSetHoverTime(NULL_HOVER_TIME)
} }
this.setState({isHoveringThisGraph: false}) this.setState({isHoveringThisGraph: false})
} }
handleHideLegend = e => { handleHideLegend = () => {
const {top, bottom, left, right} = this.graphRef.getBoundingClientRect()
const mouseY = e.clientY
const mouseX = e.clientX
const mouseInGraphY = mouseY <= bottom && mouseY >= top
const mouseInGraphX = mouseX <= right && mouseX >= left
const isMouseHoveringGraph = mouseInGraphY && mouseInGraphX
if (!isMouseHoveringGraph) {
this.setState({isHidden: true}) this.setState({isHidden: true})
} }
}
getLineColors = () => { getLineColors = () => {
return [...(this.props.overrideLineColors || LINE_COLORS)] return [...(this.props.overrideLineColors || LINE_COLORS)]