Uninstrument Text and RichText

[pixelzoom]

... the next time this sim is published. This will be an API change, and will require migration rules.

[zepumph]

phetsims/joist#934

[pixelzoom]

At the same time ashttps://github.com/phetsims/beers-law-lab/issues/332, I'm going to do this eagerly, to see how adding migration rules goes. @samreid @zepumph FYI.

[pixelzoom]

waterFaucetNode.waterText should remaining instrumented, because waterFaucetNode.waterText.visibleProperty is featured.

In general, if a Text/RichText has any featured child elements, consult with @arouinfar before uninstrumenting.

I also discussed associated derived string Properties with @arouinfar. If they are used to labels a control, keep them instrumented. If they are are readouts of values that available elsewhere, remove instrumentation.

[pixelzoom]

Done in the above commits.

Also note that ABSwitch element (graphScaleSwitch and graphUnitsSwitch) are now more sparsely instrumented, to conform to the decisions made in phetsims/sun#853 and phetsims/acid-base-solutions#190.

These changes should be reviewed by the PhET-iO designer before republishing.