Skip to content

Commit

Permalink
add option for focus highlight dilation around RoundButtonView, see #465
Browse files Browse the repository at this point in the history
  • Loading branch information
jessegreenberg committed Feb 20, 2019
1 parent cefa4fd commit eb1e32b
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions js/buttons/RoundButtonView.js
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,8 @@ define( function( require ) {
contentAppearanceStrategy: RoundButtonView.FadeContentWhenDisabled,

// a11y
tagName: 'button'
tagName: 'button',
focusHighlightDilation: 5 // radius dilation for circular highlight
}, options );

Node.call( this );
Expand Down Expand Up @@ -138,7 +139,7 @@ define( function( require ) {
this.pickable = null;

// a11y
this.focusHighlight = new Shape.circle( 0, 0, buttonRadius + 5 );
this.focusHighlight = new Shape.circle( 0, 0, buttonRadius + options.focusHighlightDilation );

// Mutate with the options after the layout is complete so that
// width-dependent fields like centerX will work.
Expand Down

1 comment on commit eb1e32b

@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.

Please sign in to comment.