Mirrors/webamp
1
0
Fork 0
mirror of https://github.com/captbaritone/webamp.git synced 2026-07-19 17:43:50 +00:00
Code Issues Projects Releases Wiki Activity

Fix issue with context menus inside playlist menus

Browse source
This commit is contained in:
Jordan Eldredge 2017-11-18 16:41:53 -08:00
parent 802b1564b2
commit 8a4e471a22
1 changed files with 14 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

20
js/components/PlaylistWindow/PlaylistMenu.js
View file

@ -54,20 +54,28 @@ export default class PlaylistMenu extends React.Component {
this.setState({ selected: false });
return;
}
this.setState({ selected: true });
const handleClickOut = ee => {
// If the click is _not_ inside the menu.
if (!target.contains(ee.target)) {
this.setState({ selected: false });
ee.stopPropagation();
// If we've clicked on a Context Menu spawed inside this menu, it will
// register as an external click. However, hiding the menu will remove
// the Context Menu from the DOM. Therefore, we wait until the next
// event loop to actually hide ourselves.
setTimeout(() => {
// Close the menu
this.setState({ selected: false });
}, 0);
window.document.removeEventListener("click", handleClickOut, {
capture: true
});
}
window.document.removeEventListener("click", handleClickOut, {
capture: true
});
};
window.document.addEventListener("click", handleClickOut, {
capture: true
});
this.setState({ selected: true });
}
render() {