diff --git a/js/energy-skate-park/graphs/view/EnergyPlot.js b/js/energy-skate-park/graphs/view/EnergyPlot.js index c5686d04..35b19a9f 100644 --- a/js/energy-skate-park/graphs/view/EnergyPlot.js +++ b/js/energy-skate-park/graphs/view/EnergyPlot.js @@ -182,16 +182,21 @@ define( require => { this.thermalEnergyDataSeries.addDataPoint( new PointStyledVector2( independentVariable, addedSample.thermalEnergy, pointStyle ) ); this.totalEnergyDataSeries.addDataPoint( new PointStyledVector2( independentVariable, addedSample.totalEnergy, pointStyle ) ); - // add a listener that updates opacity with the SkaterSample Property, dispose it on removal - const opacityListener = opacity => { - pointStyle.opacity = opacity; - this.invalidateDataSeriesNodes(); - }; - addedSample.opacityProperty.link( opacityListener ); + + // don't repaint 4 canvases ANY opacityProperty changes for any skaterSample, seems very + // inneficient + // // add a listener that updates opacity with the SkaterSample Property, dispose it on removal + // const opacityListener = opacity => { + // console.log( 'here' ); + // + // pointStyle.opacity = opacity; + // this.invalidateDataSeriesNodes(); + // }; + // addedSample.opacityProperty.link( opacityListener ); const removalListener = removedSample => { if ( removedSample === addedSample ) { - removedSample.opacityProperty.unlink( opacityListener ); + // removedSample.opacityProperty.unlink( opacityListener ); this.forEachDataSeries( dataSeries => dataSeries.removePointAtX( independentVariable ) ); model.skaterSamples.removeItemRemovedListener( removalListener ); }