phetsims / scenery-phet

Reusable components based on Scenery that are specific to PhET simulations.
MIT License
8 stars 6 forks source link

RulerNode should handle dynamic layout #817

Open marlitas opened 1 year ago

marlitas commented 1 year ago

While working on the Sound Waves prototype (https://github.com/phetsims/sound-waves/issues/23) we discovered that the unit label runs off the end of the ruler with stringTest=dynamic.

image

While looking through the code for RulerNode we determined that a fix for this was not within the scope of the Sound Waves prototype, however it should be addressed when Sound Waves is picked up again for redesign. It may be worth going through other sims that use the RulerNode to double check if they have this same problem as well.