TinyForwardingProperty.setTargetProperty() parameter order change

phetsims / axon

Axon provides powerful and concise models for interactive simulations, based on observable Properties and related patterns.

MIT License

10 stars 8 forks source link

Closed zepumph closed 4 months ago

zepumph commented 4 months ago

From https://github.com/phetsims/scenery/issues/1613, the targetProperty should be first in order.

zepumph commented 4 months ago

[x] targetProperty should be a read only until set, and not the other way around. The way it is means that Node type casts a bunch of annoying casts to call setTargetProperty.
[x] Default node/tandemName to null;