v0.6.0-beta.3 - Suggest to Add state to zoom event in ChartCanvas

Just wondering as a suggestion to add state to zoom or other events, in the example below, an event is triggered when the last item becomes visible on “panend” event… It would be nice to have this on “zoom” or even better a “zoomend” event… The zoom event currently just provides the mouseXY,currentCharts,currentItem

  componentDidMount() {
    var {canvas}=this.refs
    canvas.subscribe('canvasWidget', (eventType,chartProps,e)=> {
      this.eventChange(eventType,chartProps,e)
    });
  }

  /**
   * zoom,mousemove,draw,mouseleave,mouseenter,pan,panend
   * @param eventType
   * @param chartProps
   * @param e
   */
  eventChange(eventType,chartProps,e)
  {
    var {lastItemVisible}=this.state
    if (!isDefined(chartProps))
      return;
    switch (eventType){
      case 'panend':{
        var visibility= (dateAccessor(last(chartProps.plotData)) === dateAccessor(last(this.props.data)))
        if (lastItemVisible !== visibility)
        {
          //signal a change when the last item (most recent) changes visiblity
         console.log("lastItemVisible?",visibility)
        }
        break;
      }
    }
  }

 <ChartCanvas
            ref="canvas"
            ratio={ratio} width={width} height={height} type={type}
            margin={margin}
...

[rrag]

this is a little more complex than I though, how about this

https://github.com/rrag/react-stockcharts/blob/42a1695f5649d72ffda6df7fab381fdc8c92d74a/src/lib/ChartCanvas.jsx#L594

-each.callback(type, props, e);
+each.callback(type, props, { ...this.state, fullData: this.fullData }, e);

Where we add state and fullData as the 3rd argument and move e to 4th ?

[rrag]

Changed again to go with initial approach of using state as third param and event as 4th