diff --git a/ui/src/routes/devices.$id.settings.macros.tsx b/ui/src/routes/devices.$id.settings.macros.tsx index 5f08e32..88e59da 100644 --- a/ui/src/routes/devices.$id.settings.macros.tsx +++ b/ui/src/routes/devices.$id.settings.macros.tsx @@ -1,4 +1,4 @@ -import { useEffect, Fragment, useMemo, useState } from "react"; +import { useEffect, Fragment, useMemo, useState, useCallback } from "react"; import { useNavigate } from "react-router-dom"; import { LuPenLine, LuLoader, LuCopy, LuMoveRight, LuCornerDownRight, LuArrowUp, LuArrowDown } from "react-icons/lu"; @@ -35,7 +35,7 @@ export default function SettingsMacrosRoute() { } }, [initialized, loadMacros]); - const handleDuplicateMacro = async (macro: KeySequence) => { + const handleDuplicateMacro = useCallback(async (macro: KeySequence) => { if (!macro?.id || !macro?.name) { notifications.error("Invalid macro data"); return; @@ -67,9 +67,9 @@ export default function SettingsMacrosRoute() { } finally { setActionLoadingId(null); } - }; + }, [isMaxMacrosReached, macros, saveMacros, setActionLoadingId]); - const handleMoveMacro = async (index: number, direction: 'up' | 'down', macroId: string) => { + const handleMoveMacro = useCallback(async (index: number, direction: 'up' | 'down', macroId: string) => { if (!Array.isArray(macros) || macros.length === 0) { notifications.error("No macros available"); return; @@ -96,7 +96,7 @@ export default function SettingsMacrosRoute() { } finally { setActionLoadingId(null); } - }; + }, [macros, saveMacros, setActionLoadingId]); const MacroList = useMemo(() => (