diff --git a/css/main-window.css b/css/main-window.css index 65a94947..e9557a45 100755 --- a/css/main-window.css +++ b/css/main-window.css @@ -80,7 +80,7 @@ left: 0px; } #winamp2-js #title-bar #option:active #context-menu, -#winamp2-js #title-bar #option.selected #context-menu { +#winamp2-js #title-bar #option .selected #context-menu { display: block; } diff --git a/js/components/ContextMenu.js b/js/components/ContextMenu.js index 5509a4eb..546010ff 100644 --- a/js/components/ContextMenu.js +++ b/js/components/ContextMenu.js @@ -48,11 +48,7 @@ export class ContextMenu extends React.Component { render() { return ( -
+
@@ -63,6 +59,5 @@ export class ContextMenu extends React.Component { ContextMenu.propTypes = { closeMenu: React.PropTypes.func.isRequired, - toggleMenu: React.PropTypes.func.isRequired, children: React.PropTypes.any.isRequired }; diff --git a/js/components/MainContextMenu.js b/js/components/MainContextMenu.js index 963eacc3..89526cd9 100644 --- a/js/components/MainContextMenu.js +++ b/js/components/MainContextMenu.js @@ -14,11 +14,7 @@ const SKINS = [ ]; const MainContextMenu = props => - + ({ console.log("close"); dispatch({ type: CLOSE_CONTEXT_MENU }); }, - toggleMenu: e => { - dispatch({ type: TOGGLE_CONTEXT_MENU }); - // TODO: Consider binding to a ref instead. - // https://stackoverflow.com/a/24421834 - e.nativeEvent.stopImmediatePropagation(); - }, openFileDialog: () => dispatch(openFileDialog(ownProps.fileInput)), setSkin: filename => dispatch(setSkinFromFilename(filename)) }); diff --git a/js/components/MainWindow.js b/js/components/MainWindow.js index 8a170715..373b18d3 100644 --- a/js/components/MainWindow.js +++ b/js/components/MainWindow.js @@ -24,7 +24,7 @@ import Time from "./Time"; import Visualizer from "./Visualizer"; import Volume from "./Volume"; -import { SET_FOCUSED_WINDOW } from "../actionTypes"; +import { SET_FOCUSED_WINDOW, TOGGLE_CONTEXT_MENU } from "../actionTypes"; import { loadFileFromReference } from "../actionCreators"; @@ -91,7 +91,9 @@ export class MainWindow extends React.Component { >
Loading...
- +
+ +
@@ -144,4 +146,14 @@ const mapStateToProps = state => { } = state; return { status, loading, doubled, shade, closed, llama, working, focused }; }; -export default connect(mapStateToProps)(MainWindow); + +const mapDispatchToProps = dispatch => ({ + toggleMenu: e => { + dispatch({ type: TOGGLE_CONTEXT_MENU }); + // TODO: Consider binding to a ref instead. + // https://stackoverflow.com/a/24421834 + e.nativeEvent.stopImmediatePropagation(); + }, + dispatch +}); +export default connect(mapStateToProps, mapDispatchToProps)(MainWindow);