Mirrors/webamp
1
0
Fork 0
mirror of https://github.com/captbaritone/webamp.git synced 2026-08-03 07:23:55 +00:00
Code Issues Projects Releases Wiki Activity

Fix context menus in Firefox

Browse source 
This is a huge mess of timing.
This commit is contained in:
Jordan Eldredge 2018-08-14 19:24:51 -07:00
parent 24a618bf0b
commit 32098e3b62
1 changed files with 20 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

27
js/components/ContextMenuWrapper.js
View file

@ -12,19 +12,31 @@ export default class ContextMenuWraper extends React.Component {
};
this._handleRightClick = this._handleRightClick.bind(this);
this._handleGlobalClick = this._handleGlobalClick.bind(this);
this._handleGlobalRightClick = this._handleGlobalRightClick.bind(this);
}
componentWillUnmount() {
document.removeEventListener("click", this._handleGlobalClick);
this._closeMenu();
}
_handleGlobalClick() {
this.setState({
selected: false,
offsetTop: null,
offsetLeft: null
});
_closeMenu() {
this.setState({ selected: false, offsetTop: null, offsetLeft: null });
document.removeEventListener("click", this._handleGlobalClick);
document.body.removeEventListener(
"contextmenu",
this._handleGlobalRightClick
);
}
_handleGlobalRightClick() {
this._closeMenu();
}
_handleGlobalClick(e) {
if (e.button === 2) {
return;
}
this._closeMenu();
}
_handleRightClick(e) {
@ -39,6 +51,7 @@ export default class ContextMenuWraper extends React.Component {
// Even if you right click multiple times before closeing,
// we should only end up with one global listener.
document.addEventListener("click", this._handleGlobalClick);
document.body.addEventListener("contextmenu", this._handleGlobalRightClick);
e.preventDefault();
e.stopPropagation();
}