Skip to content

Commit

Permalink
manually dispose KeyboardHelpSectionRows, phetsims/scenery-phet#769
Browse files Browse the repository at this point in the history
  • Loading branch information
zepumph committed Apr 4, 2023
1 parent 4352cb8 commit f8a264d
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions js/common/view/GOKeyboardHelpContent.ts
Original file line number Diff line number Diff line change
Expand Up @@ -101,8 +101,7 @@ class MoveDraggableItemsSection extends KeyboardHelpSection {

this.disposeMoveDraggableItemsSection = () => {
icons.forEach( icon => icon.dispose() );
//TODO https://github.com/phetsims/scenery-phet/issues/769 uncomment when KeyboardHelpSection no longer disposes of content
//rows.forEach( row => row.dispose() );
rows.forEach( row => row.dispose() );
};
}

Expand Down Expand Up @@ -148,8 +147,7 @@ class RulerAndMarkerControlsSection extends KeyboardHelpSection {

this.disposeRulerAndMarkerControlsSection = () => {
keyNodes.forEach( keyNode => keyNode.dispose() );
//TODO https://github.com/phetsims/scenery-phet/issues/769 uncomment when KeyboardHelpSection no longer disposes of content
//rows.forEach( row => row.dispose() );
rows.forEach( row => row.dispose() );
};
}

Expand Down

0 comments on commit f8a264d

Please sign in to comment.