refactor: update keyboard event handling to use string literals and remove unused state in EventForm

Author: tyler-dane

packages/web/src/views/Day/components/Agenda/Agenda.tsx

@@ -65,7 +65,7 @@ export function Agenda() {
   });
 
   const onEnterKey = useCallback((e: React.KeyboardEvent<HTMLDivElement>) => {
-    if (e.key === Key.Enter) {
+    if (e.key === "Enter") {
       e.preventDefault();
       e.stopPropagation();
       timedEventsGridRef.current?.click();

packages/web/src/views/Forms/EventForm/EventForm.tsx

@@ -62,7 +62,6 @@ export const EventForm: React.FC<Omit<FormProps, "category">> = memo(
       label: "1 AM",
       value: "01:00 AM",
     });
-    const [_isShiftKeyPressed, setIsShiftKeyPressed] = useState(false);
     const [isFormOpen, setIsFormOpen] = useState(false);
     const [isEndDatePickerOpen, setIsEndDatePickerOpen] = useState(false);
     const [isStartDatePickerOpen, setIsStartDatePickerOpen] = useState(false);
@@ -76,36 +75,6 @@ export const EventForm: React.FC<Omit<FormProps, "category">> = memo(
 
     const descriptionRef = useRef<HTMLTextAreaElement>(null);
 
-    /*********
-     * Effects
-     *********/
-
-    const keyDownHandler = useCallback(
-      (e: globalThis.KeyboardEvent) => {
-        if (e.key === "Shift") {
-          setIsShiftKeyPressed(true);
-        }
-      },
-      [_onClose],
-    );
-
-    const keyUpHandler = useCallback((e: globalThis.KeyboardEvent) => {
-      if (e.key === "Shift") {
-        setIsShiftKeyPressed(false);
-      }
-    }, []);
-
-    useEffect(() => {
-      window.addEventListener("keydown", keyDownHandler);
-      window.addEventListener("keyup", keyUpHandler);
-
-      return () => {
-        window.removeEventListener("keydown", keyDownHandler);
-        window.removeEventListener("keyup", keyUpHandler);
-      };
-      // eslint-disable-next-line react-hooks/exhaustive-deps
-    }, []);
-
     useEffect(() => {
       setEvent(event || {});