phetsims / fourier-making-waves

"Fourier: Making Waves" is an educational simulation in HTML5, by PhET Interactive Simulations.
GNU General Public License v3.0
6 stars 3 forks source link

Implement dispose for keyboard-help content #227

Closed pixelzoom closed 1 year ago

pixelzoom commented 1 year ago

Related to https://github.com/phetsims/scenery-phet/issues/769 ...

This sim currently does not dispose of keyboard-help content, and currently has dispose implemented like this:

  dispose() {
    assert && assert( false, 'dispose is not supported, exists for the lifetime of the sim' );
    super.dispose();
  }

The sim will therefore fail in the PhET-iO State Wrapper, because it creates and then immediately disposes PhetioCapsules on startup.

Implement dispose for these classes:

pixelzoom commented 1 year ago

This is not currently a priority because this sim has not had a PhET-iO design, and there are no specific plans for a PhET-iO version or release. So I'm leaving this issue unassigned.

pixelzoom commented 1 year ago

In the above commit, I implemented dispose for keyboard help content. Using the process and patch in https://github.com/phetsims/scenery-phet/issues/769#issuecomment-1301473507, memory looks good:

Closing.