From 49cd724470d03f32a2cb83c2767dcf212dc01222 Mon Sep 17 00:00:00 2001
From: marcobaobao <marco.piovanello@gmail.com>
Date: Sun, 8 May 2022 11:25:41 +0200
Subject: [PATCH] Dark theme integration

---
 frontend/src/App.tsx      | 38 ++++++++++++-------
 frontend/src/Settings.tsx | 78 +++++++++++++++++++++------------------
 2 files changed, 67 insertions(+), 49 deletions(-)

diff --git a/frontend/src/App.tsx b/frontend/src/App.tsx
index 1d7d5cb..95d03d1 100755
--- a/frontend/src/App.tsx
+++ b/frontend/src/App.tsx
@@ -1,4 +1,4 @@
-import React, { useEffect, useState } from "react"
+import React, { useEffect, useMemo, useState } from "react"
 import { ThemeProvider } from "@emotion/react";
 import {
     Badge,
@@ -13,7 +13,13 @@ import {
 } from "@mui/material"
 import MuiDrawer from '@mui/material/Drawer';
 import MuiAppBar, { AppBarProps as MuiAppBarProps } from '@mui/material/AppBar';
-import { ChevronLeft, Dashboard, Menu, Settings as SettingsIcon, SettingsEthernet, Storage } from "@mui/icons-material";
+import {
+    ChevronLeft,
+    Dashboard,
+    Menu, Settings as SettingsIcon,
+    SettingsEthernet,
+    Storage,
+} from "@mui/icons-material";
 import ListItemButton from '@mui/material/ListItemButton';
 import {
     BrowserRouter as Router,
@@ -25,9 +31,9 @@ import Home from "./Home";
 import Settings from "./Settings";
 import { io } from "socket.io-client";
 import { RootState, store } from './stores/store';
-import { Provider, useSelector } from "react-redux";
+import { Provider, useDispatch, useSelector } from "react-redux";
+import { setTheme } from "./features/settings/settingsSlice";
 
-const theme = createTheme();
 
 const drawerWidth: number = 240;
 
@@ -85,13 +91,23 @@ function AppContent() {
     const [open, setOpen] = useState(false);
     const [freeDiskSpace, setFreeDiskSpace] = useState('');
 
+    const settings = useSelector((state: RootState) => state.settings)
+    const status = useSelector((state: RootState) => state.status)
+
+    const mode = settings.theme
+
+    const theme = useMemo(() =>
+        createTheme({
+            palette: {
+                mode,
+            },
+        }), [settings.theme]
+    );
+
     const toggleDrawer = () => {
         setOpen(!open);
     };
 
-    const settings = useSelector((state: RootState) => state.settings)
-    const status = useSelector((state: RootState) => state.status)
-
     /* Get disk free space */
     useEffect(() => {
         socket.on('free-space', (res: string) => {
@@ -175,7 +191,7 @@ function AppContent() {
                             <Link to={'/'} style={
                                 {
                                     textDecoration: 'none',
-                                    color: '#222222'
+                                    color: mode === 'dark' ? '#ffffff' : '#000000DE'
                                 }
                             }>
                                 <ListItemButton disabled={status.downloading}>
@@ -188,7 +204,7 @@ function AppContent() {
                             <Link to={'/settings'} style={
                                 {
                                     textDecoration: 'none',
-                                    color: '#222222'
+                                    color: mode === 'dark' ? '#ffffff' : '#000000DE'
                                 }
                             }>
                                 <ListItemButton disabled={status.downloading}>
@@ -203,10 +219,6 @@ function AppContent() {
                     <Box
                         component="main"
                         sx={{
-                            backgroundColor: (theme) =>
-                                theme.palette.mode === 'light'
-                                    ? theme.palette.grey[100]
-                                    : theme.palette.grey[900],
                             flexGrow: 1,
                             height: '100vh',
                             overflow: 'auto',
diff --git a/frontend/src/Settings.tsx b/frontend/src/Settings.tsx
index 9240c72..df0f612 100644
--- a/frontend/src/Settings.tsx
+++ b/frontend/src/Settings.tsx
@@ -17,10 +17,10 @@ import {
     TextField,
     Typography
 } from "@mui/material";
-import React, { useEffect, useState } from "react";
+import React, { useState } from "react";
 import { useDispatch, useSelector } from "react-redux";
 import { Socket } from "socket.io-client";
-import { LanguageUnion, setCliArgs, setLanguage, setServerAddr } from "./features/settings/settingsSlice";
+import { LanguageUnion, setCliArgs, setLanguage, setServerAddr, setTheme, ThemeUnion } from "./features/settings/settingsSlice";
 import { alreadyUpdated, updated } from "./features/status/statusSlice";
 import { RootState } from "./stores/store";
 import { validateDomain, validateIP } from "./utils";
@@ -34,25 +34,7 @@ export default function Settings({ socket }: Props) {
     const status = useSelector((state: RootState) => state.status)
     const dispatch = useDispatch()
 
-    const [halt, setHalt] = useState(false);
     const [invalidIP, setInvalidIP] = useState(false);
-    const [updatedBin, setUpdatedBin] = useState(false);
-    const [freeDiskSpace, setFreeDiskSpace] = useState('');
-
-    /* Handle yt-dlp update success */
-    useEffect(() => {
-        socket.on('updated', () => {
-            setUpdatedBin(true)
-            setHalt(false)
-        })
-    }, [])
-
-    /* Get disk free space */
-    useEffect(() => {
-        socket.on('free-space', (res: string) => {
-            setFreeDiskSpace(res)
-        })
-    }, [])
 
     /**
      * Update the server ip address state and localstorage whenever the input value changes.  
@@ -75,10 +57,17 @@ export default function Settings({ socket }: Props) {
     /**
      * Language toggler handler 
      */
-    const handleLanguageChage = (event: SelectChangeEvent<LanguageUnion>) => {
+    const handleLanguageChange = (event: SelectChangeEvent<LanguageUnion>) => {
         dispatch(setLanguage(event.target.value as LanguageUnion));
     }
 
+    /**
+     * Language toggler handler 
+     */
+    const handleThemeChange = (event: SelectChangeEvent<ThemeUnion>) => {
+        dispatch(setTheme(event.target.value as ThemeUnion));
+    }
+
     /**
      * Send via WebSocket a message in order to update the yt-dlp binary from server
      */
@@ -115,21 +104,38 @@ export default function Settings({ socket }: Props) {
                                 }}
                                 sx={{ mb: 2 }}
                             />
-                            <FormControl fullWidth>
-                                <InputLabel id="demo-simple-select-label">Language</InputLabel>
-                                <Select
-                                    defaultValue={settings.language}
-                                    label="Language"
-                                    onChange={handleLanguageChage}
-                                >
-                                    <MenuItem value="english">English</MenuItem>
-                                    <MenuItem value="spanish">Spanish</MenuItem>
-                                    <MenuItem value="italian">Italian</MenuItem>
-                                    <MenuItem value="chinese">Chinese</MenuItem>
-                                    <MenuItem value="russian">Russian</MenuItem>
-                                    <MenuItem value="korean">Korean</MenuItem>
-                                </Select>
-                            </FormControl>
+                            <Grid container spacing={2}>
+                                <Grid item xs={12} md={6}>
+                                    <FormControl fullWidth>
+                                        <InputLabel id="demo-simple-select-label">Language</InputLabel>
+                                        <Select
+                                            defaultValue={settings.language}
+                                            label="Language"
+                                            onChange={handleLanguageChange}
+                                        >
+                                            <MenuItem value="english">English</MenuItem>
+                                            <MenuItem value="spanish">Spanish</MenuItem>
+                                            <MenuItem value="italian">Italian</MenuItem>
+                                            <MenuItem value="chinese">Chinese</MenuItem>
+                                            <MenuItem value="russian">Russian</MenuItem>
+                                            <MenuItem value="korean">Korean</MenuItem>
+                                        </Select>
+                                    </FormControl>
+                                </Grid>
+                                <Grid item xs={12} md={6}>
+                                    <FormControl fullWidth>
+                                        <InputLabel>Theme</InputLabel>
+                                        <Select
+                                            defaultValue={settings.theme}
+                                            label="Theme"
+                                            onChange={handleThemeChange}
+                                        >
+                                            <MenuItem value="light">Light</MenuItem>
+                                            <MenuItem value="dark">Dark</MenuItem>
+                                        </Select>
+                                    </FormControl>
+                                </Grid>
+                            </Grid>
                             <FormControlLabel
                                 control={
                                     <Switch