Skip to content

Commit 6aa7822

Browse files
authored
Check the value of event.button only if mouseButton is explicitely specified (PR #3)
1 parent 3ed9a25 commit 6aa7822

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

src/ContextMenuTrigger.js

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ export default class ContextMenuTrigger extends Component {
1717
posX: PropTypes.number,
1818
posY: PropTypes.number,
1919
renderTag: PropTypes.elementType,
20+
// 0 is left click, 2 is right click
2021
mouseButton: PropTypes.number,
2122
disableIfShiftIsPressed: PropTypes.bool
2223
};
@@ -29,7 +30,7 @@ export default class ContextMenuTrigger extends Component {
2930
renderTag: 'div',
3031
posX: 0,
3132
posY: 0,
32-
mouseButton: 2, // 0 is left click, 2 is right click
33+
mouseButton: null,
3334
disableIfShiftIsPressed: false
3435
};
3536

@@ -89,14 +90,16 @@ export default class ContextMenuTrigger extends Component {
8990
}
9091

9192
handleContextMenu = (event) => {
92-
if (event.button === this.props.mouseButton) {
93+
const { mouseButton } = this.props;
94+
if (mouseButton === null || event.button === mouseButton) {
9395
this.handleContextClick(event);
9496
}
9597
callIfExists(this.props.attributes.onContextMenu, event);
9698
}
9799

98100
handleMouseClick = (event) => {
99-
if (event.button === this.props.mouseButton) {
101+
const { mouseButton } = this.props;
102+
if (mouseButton === null || event.button === mouseButton) {
100103
this.handleContextClick(event);
101104
}
102105
callIfExists(this.props.attributes.onClick, event);

0 commit comments

Comments
 (0)