Check for options change in ComponentWillReceiveProps

[jgliner]

Currently, options won't update along with the chart if new props are passed. This is especially an issue when updating charts dynamically (e.g. changing the yAxes label from a dropdown menu). There may be a slightly more elegant way to check for changes in user-defined options, but this check should not negatively affect performance.

[Jareechang]

hi @jgliner, would it possible to redraw when you update yAxes option on your side ?

(yAxesChanged 
   ? <LineChart redraw />
   : <LineChart />) 

I want to make sure we're minimizing regressions as we currently do not have any tests (will add some soon).

Please let me know if that works for you. Thanks

[Jareechang]

Hi @jgliner, sorry I mis-understood. I see what you are talking about.

If this is the case, I prefer that we use react's core utility for this case. Please see below for example.

npm i fbjs

var shallowEqual = require('fbjs/lib/shallowEqual');
var optsChange = !shallowEqual(nextProps.options, this.props.options);

Please update PR when you get a chance. Thank you.