TODO: what if the chart area changes

[pixelzoom]

In DemoCanvasLinePlot.js and DemoLinePlot.js:

// TODO https://github.com/phetsims/bamboo/issues/15 what if the chart area changes, then clip needs to change
clipArea: chartRectangle.getShape(),

[pixelzoom]

If the view dimensions of the ChartModel were to change, by using setWidth or setHeight, then you'd adjust the clipArea like this:

const clippedNode = new Node( {
   clipArea: chartRectangle.getShape(),
   ...
} );

chartRectangle.localBoundsProperty.lazyLink( localBounds => {
  clippedNode.clipArea = chartRectangle.getShape();
} );

That's pretty standard/straightforward, so I think we can simply delete these TODOs.

@samreid your thoughts?

[samreid]

The main question for me is if we should provide a bamboo type like ChartClipNode which automatically does that.

[pixelzoom]

I can't recall implementing a sim chart whose view bounds change, so I doubt that ChartClipNode would get much use. But feel free to add it now if you'd like, or wait until it's needed. Up to you...

[samreid]

It seems most appropriate to develop this in context, if and when we need it. Unassigning for now.