From ba72217e21b787c1b09b92dc3255abc5027ede51 Mon Sep 17 00:00:00 2001
From: Robin Fernandes <robin@soal.org>
Date: Fri, 11 Aug 2023 09:09:01 +1000
Subject: [PATCH] #197, #198: darkmode and minor UX improvement to keyframe
 dialogs.

---
 src/components/KeyframeDialogs.tsx | 36 ++++++++++++++++++++++--------
 1 file changed, 27 insertions(+), 9 deletions(-)

diff --git a/src/components/KeyframeDialogs.tsx b/src/components/KeyframeDialogs.tsx
index 1f73ee7..cd9c706 100644
--- a/src/components/KeyframeDialogs.tsx
+++ b/src/components/KeyframeDialogs.tsx
@@ -25,6 +25,8 @@ import { frameToXAxisType, xAxisTypeToFrame } from '../utils/maths';
 import * as _ from 'lodash';
 import { parse } from '../parseq-lang/parseq-lang-parser';
 import { InvocationContext } from '../parseq-lang/parseq-lang-ast';
+import { SupportedColorScheme, experimental_extendTheme as extendTheme, useColorScheme } from "@mui/material/styles";
+import { themeFactory } from '../theme';
 
 interface AddKeyframesDialogProps {
     keyframes: ParseqKeyframe[];
@@ -53,6 +55,11 @@ export const AddKeyframesDialog: FC<AddKeyframesDialogProps> = ({
     const [start, setStart] = useState("0");
     const [end, setEnd] = useState(String(lastFrame));
 
+    const theme = extendTheme(themeFactory());
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+    const {colorScheme, setColorScheme }  = useColorScheme();
+    const palette = theme.colorSchemes[(colorScheme||'light') as SupportedColorScheme].palette;
+
     function handleCloseAddKeyframesDialog(event: any): void {
         setOpenAddKeyframesDialog(false);
         if (event?.target?.id === "add") {
@@ -181,17 +188,17 @@ export const AddKeyframesDialog: FC<AddKeyframesDialogProps> = ({
                 </TabPanel>
                 <DialogContentText overflow={"wrap"}>
                     {framesToAdd.length > 0 && <>
-                        <Typography fontSize={"0.75em"} fontStyle={{ color: "darkgreen" }}>
+                        <Typography fontSize={"0.75em"} fontStyle={{ color:  palette.success.main }}>
                             Will add {framesToAdd.length} keyframe(s):
                         </Typography>
-                        <Typography fontFamily={"monospace"} fontSize={"0.75em"} fontStyle={{ color: "darkgreen" }}>
+                        <Typography fontFamily={"monospace"} fontSize={"0.75em"} fontStyle={{ color:  palette.success.main }}>
                             • Frames: {framesToAdd.sort((a, b) => a - b).join(', ')}<br />
                             • Seconds: {framesToAdd.sort((a, b) => a - b).map(f => frameToXAxisType(f, "seconds", fps, bpm)).join(', ')}<br />
                             • Beats: {framesToAdd.sort((a, b) => a - b).map(f => frameToXAxisType(f, "beats", fps, bpm)).join(', ')}<br />
                         </Typography>
                     </>}
                     {framesToAdd.length <= 0 &&
-                        <Typography fontSize={"0.75em"} fontStyle={{ color: "darkgreen" }}>
+                        <Typography fontSize={"0.75em"} fontStyle={{ color:  palette.success.main }}>
                             No frames to add.
                         </Typography>
                     }
@@ -240,6 +247,12 @@ export const DeleteKeyframesDialog: FC<DeleteKeyframesDialogProps> = ({
     const [deleteMatchRegex, setDeleteMatchRegex] = useState('');
     const [deleteMatchFieldType, setDeleteMatchFieldType] = useState<'value' | 'interpolation'>('value');
 
+    const theme = extendTheme(themeFactory());
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+    const {colorScheme, setColorScheme }  = useColorScheme();
+    const palette = theme.colorSchemes[(colorScheme||'light') as SupportedColorScheme].palette;
+
+
     useEffect(() => {
         setFramesToDeleteList(initialFramesToDelete.join(', '));
     }, [initialFramesToDelete]);
@@ -359,10 +372,10 @@ export const DeleteKeyframesDialog: FC<DeleteKeyframesDialogProps> = ({
                 </TabPanel>
                 <DialogContentText overflow={"wrap"}>
                     {framesToDelete.length > 0 && <>
-                        <Typography fontSize={"0.75em"} fontStyle={{ color: "firebrick" }}>
+                        <Typography fontSize={"0.75em"} fontStyle={{ color: palette.warning.dark }}>
                             Will delete {framesToDelete.length} keyframe(s):
                         </Typography>
-                        <Typography fontFamily={"monospace"} fontSize={"0.75em"} fontStyle={{ color: "firebrick" }}>
+                        <Typography fontFamily={"monospace"} fontSize={"0.75em"} fontStyle={{ color: palette.warning.dark }}>
                             • Frames: {framesToDelete.sort((a, b) => a - b).join(', ')}<br />
                             • Seconds: {framesToDelete.sort((a, b) => a - b).map(f => frameToXAxisType(f, "seconds", fps, bpm)).join(', ')}<br />
                             • Beats: {framesToDelete.sort((a, b) => a - b).map(f => frameToXAxisType(f, "beats", fps, bpm)).join(', ')}<br />
@@ -418,6 +431,11 @@ export const BulkEditDialog: FC<BulkEditDialogProps> = ({
     const [fieldTypeToUpdate, setFieldTypeToUpdate] = useState<'value' | 'interpolation'>('value');
     const [newValue, setNewValue] = useState('');
 
+    const theme = extendTheme(themeFactory());
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+    const {colorScheme, setColorScheme }  = useColorScheme();
+    const palette = theme.colorSchemes[(colorScheme||'light') as SupportedColorScheme].palette;
+
     const framesToEdit = useMemo(() => {
         if (matchRegex) {
             return findMatchingKeyframes(keyframes, matchRegex, matchMethod, matchField,
@@ -565,17 +583,17 @@ export const BulkEditDialog: FC<BulkEditDialogProps> = ({
                 </Stack>
                 <DialogContentText>
                     {framesToEdit.length > 0 && <>
-                        <Typography fontSize={"0.75em"} fontStyle={{ color: "darkgreen" }}>
+                        <Typography fontSize={"0.75em"} fontStyle={{ color:  palette.success.main }}>
                             Will edit {framesToEdit.length} keyframe(s):
                         </Typography>
-                        <Typography fontFamily={"monospace"} fontSize={"0.75em"} fontStyle={{ color: "darkgreen" }}>
+                        <Typography fontFamily={"monospace"} fontSize={"0.75em"} fontStyle={{ color:  palette.success.main }}>
                             • Frames: {framesToEdit.sort((a, b) => a - b).join(', ')}<br />
                             • Seconds: {framesToEdit.sort((a, b) => a - b).map(f => frameToXAxisType(f, "seconds", fps, bpm)).join(', ')}<br />
                             • Beats: {framesToEdit.sort((a, b) => a - b).map(f => frameToXAxisType(f, "beats", fps, bpm)).join(', ')}<br />
                         </Typography>
                     </>}
                     {framesToEdit.length <= 0 &&
-                        <Typography fontSize={"0.75em"} fontStyle={{ color: "darkgreen" }}>
+                        <Typography fontSize={"0.75em"} fontStyle={{ color: palette.success.main }}>
                             No frames to edit.
                         </Typography>
                     }
@@ -587,7 +605,7 @@ export const BulkEditDialog: FC<BulkEditDialogProps> = ({
                     Cancel
                 </Button>
                 <Button disabled={framesToEdit.length === 0} variant="contained" id="edit" onClick={handleCloseDialog}>
-                    🖌️ Bulk Edit
+                    🖌️ Apply
                 </Button>
             </DialogActions>
         </Dialog>