phetsims / griddle

Dynamic charting library built with Scenery
MIT License
2 stars 4 forks source link

delete deprecated classes in griddle? #69

Closed pixelzoom closed 4 years ago

pixelzoom commented 4 years ago

Noted while working on https://github.com/phetsims/tasks/issues/1044.

XYPlotNode and XYDataSeriesNode are both annotated as @deprecated. They are only used in the griddle demo. Is there any reason not to delete them?

Unfortunately I noticed this after spending the time to convert XYDataSeriesNode to ES6. Fortunately I noticed before converting XYPlotNode to ES6.

samreid commented 4 years ago

Yes, I'll delete them as part of https://github.com/phetsims/griddle/issues/63