Skip to content

Commit

Permalink
remove custom layoutStrategy of KeyboardHelpDialog, see #435
Browse files Browse the repository at this point in the history
  • Loading branch information
jessegreenberg committed Jan 17, 2019
1 parent 3a787db commit 86a3d9f
Showing 1 changed file with 1 addition and 11 deletions.
12 changes: 1 addition & 11 deletions js/KeyboardHelpDialog.js
Original file line number Diff line number Diff line change
Expand Up @@ -45,17 +45,7 @@ define( function( require ) {
titleAlign: 'center',
fill: 'rgb( 214, 237, 249 )',
ySpacing: 15,
tandem: Tandem.required,

// the width of the KeyboardHelpDialog is constrained to the simulation bounds
layoutStrategy: function( dialog, simBounds, screenBounds, scale ) {

// empirically determined to have some margin relative to max width
dialog.maxWidth = ( simBounds.width / scale ) * 0.90;

// this is the default layout strategy in Dialog
dialog.center = simBounds.center.times( 1.0 / scale );
}
tandem: Tandem.required
}, options );

// title
Expand Down

1 comment on commit 86a3d9f

@jessegreenberg
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was meant for phetsims/sun#435

Please sign in to comment.