Skip to content

Commit d42b15a

Browse files
authored
Merge pull request #151 from dahjelle/fix-execute-button
Fixes #150.
2 parents 428e0da + 9cfb3a4 commit d42b15a

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

src/components/ExecuteButton.js

+7-1
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,13 @@ export class ExecuteButton extends React.Component {
111111
} else {
112112
document.removeEventListener('mouseup', onMouseUp);
113113
onMouseUp = null;
114-
this.setState({ optionsOpen: false });
114+
const isOptionsMenuClicked = (
115+
downTarget.parentNode.compareDocumentPosition(upEvent.target) &
116+
Node.DOCUMENT_POSITION_CONTAINED_BY
117+
);
118+
if (!isOptionsMenuClicked) { // menu calls setState if it was clicked
119+
this.setState({ optionsOpen: false });
120+
}
115121
}
116122
};
117123

0 commit comments

Comments
 (0)