From 33cd6a5c276d9d0f3b4281dd9fddfdff8303aa41 Mon Sep 17 00:00:00 2001
From: Jordan Eldredge <jordan@jordaneldredge.com>
Date: Mon, 23 Dec 2019 18:12:55 -0800
Subject: [PATCH] Use hooks in App.tsx

---
 js/components/App.tsx | 73 +++++++++----------------------------------
 js/selectors.ts       |  8 +++++
 2 files changed, 22 insertions(+), 59 deletions(-)

diff --git a/js/components/App.tsx b/js/components/App.tsx
index da92090a..683113cc 100644
--- a/js/components/App.tsx
+++ b/js/components/App.tsx
@@ -5,15 +5,7 @@ import React, {
   useCallback,
 } from "react";
 import ReactDOM from "react-dom";
-import { connect } from "react-redux";
-import {
-  WindowId,
-  AppState,
-  Dispatch,
-  Size,
-  MediaStatus,
-  FilePicker,
-} from "../types";
+import { FilePicker } from "../types";
 import { WINDOWS } from "../constants";
 import * as Selectors from "../selectors";
 import * as Actions from "../actionCreators";
@@ -28,43 +20,27 @@ import EqualizerWindow from "./EqualizerWindow";
 import Skin from "./Skin";
 
 import "../../css/webamp.css";
-import { WebampWindow } from "../reducers/windows";
 import Media from "../media";
+import { useTypedSelector, useActionCreator } from "../hooks";
 
-interface StateProps {
-  visualizerStyle: string;
-  status: MediaStatus;
-  closed: boolean;
-  // TODO: Get only the info we really need
-  genWindowsInfo: { [windowId: string]: WebampWindow };
-  zIndex: number;
-}
-
-interface DispatchProps {
-  closeWindow(id: WindowId): void;
-  browserWindowSizeChanged(size: Size): void;
-  clearFocus(): void;
-}
-
-interface OwnProps {
+interface Props {
   filePickers: FilePicker[];
   media: Media;
 }
 
-type Props = StateProps & DispatchProps & OwnProps;
-
 /**
  * Constructs the windows to render
  */
-function App({
-  zIndex,
-  browserWindowSizeChanged,
-  closed,
-  clearFocus,
-  media,
-  genWindowsInfo,
-  filePickers,
-}: Props) {
+export default function App({ media, filePickers }: Props) {
+  const closed = useTypedSelector(Selectors.getClosed);
+  const genWindowsInfo = useTypedSelector(Selectors.getGenWindows);
+  const zIndex = useTypedSelector(Selectors.getZIndex);
+
+  const browserWindowSizeChanged = useActionCreator(
+    Actions.browserWindowSizeChanged
+  );
+  const setFocusedWindow = useActionCreator(Actions.setFocusedWindow);
+
   const [webampNode] = useState(() => {
     const node = document.createElement("div");
     node.id = "webamp";
@@ -142,7 +118,7 @@ function App({
 
   const handleBlur = (e: React.FocusEvent<HTMLElement>) => {
     if (!e.currentTarget.contains(e.relatedTarget as Element)) {
-      clearFocus();
+      setFocusedWindow(null);
     }
   };
 
@@ -164,24 +140,3 @@ function App({
     webampNode
   );
 }
-
-const mapStateToProps = (state: AppState): StateProps => {
-  return {
-    visualizerStyle: Selectors.getVisualizerStyle(state),
-    status: state.media.status,
-    closed: state.display.closed,
-    genWindowsInfo: state.windows.genWindows,
-    zIndex: state.display.zIndex,
-  };
-};
-
-const mapDispatchToProps = (dispatch: Dispatch): DispatchProps => {
-  return {
-    closeWindow: (id: WindowId) => dispatch(Actions.closeWindow(id)),
-    browserWindowSizeChanged: (size: Size) =>
-      dispatch(Actions.browserWindowSizeChanged(size)),
-    clearFocus: () => dispatch(Actions.setFocusedWindow(null)),
-  };
-};
-
-export default connect(mapStateToProps, mapDispatchToProps)(App);
diff --git a/js/selectors.ts b/js/selectors.ts
index ae6ef6d7..09c044cb 100644
--- a/js/selectors.ts
+++ b/js/selectors.ts
@@ -434,6 +434,10 @@ export function getLlamaMode(state: AppState) {
   return state.display.llama;
 }
 
+export function getZIndex(state: AppState): number {
+  return state.display.zIndex;
+}
+
 // TODO: This is poorly memoized. It invalidates when a window moves.
 export const getWindowSizes = createSelector(
   getGenWindows,
@@ -681,6 +685,10 @@ export function getRandomizePresets(state: AppState): boolean {
   return state.milkdrop.randomize;
 }
 
+export function getClosed(state: AppState): boolean {
+  return state.display.closed;
+}
+
 export function getSkinImages(state: AppState): SkinImages {
   return state.display.skinImages;
 }