Skip to content

Commit

Permalink
Use role="button" instead of href in MenuItem (#97)
Browse files Browse the repository at this point in the history
* bugfix: dont use a href
* PR 97 review: use role=button
* PR 97 review: alphabetize props for linter
  • Loading branch information
John Blossom authored and ericgio committed Oct 24, 2016
1 parent 13edccb commit 399f5aa
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/MenuItem.react.js
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ const MenuItem = React.createClass({
'active': active,
'disabled': disabled,
}, className)}>
<a href="#" onClick={this._handleClick}>
<a onClick={this._handleClick} role="button">
{children}
</a>
</li>
Expand Down

0 comments on commit 399f5aa

Please sign in to comment.