Can we get rid of FloatingLayout and just use visibleBoundsProperty directly?

phetsims / bending-light

"Bending Light" is an educational simulation in HTML5, by PhET Interactive Simulations.

GNU General Public License v3.0

8 stars 8 forks source link

Closed samreid closed 8 years ago

samreid commented 8 years ago

It looks like FloatingLayout is just adding convenience methods and constants for insets.

samreid commented 8 years ago

We could fix this in #351

samreid commented 8 years ago

I reviewed FloatingLayout.js and determined that it is performing the following important functions:

Letting objects move outside the layoutBounds but not too far (not always all the way to the visibleBounds)
Convenience methods for laying out many objects at once
Encapsulation of insets and other layout measures

It should remain.