Fix lint errors

This commit is contained in:
Adam Shiervani 2025-04-16 22:03:43 +02:00
parent df74cb111d
commit 7cd3d32926
8 changed files with 459 additions and 336 deletions

View File

@ -1,7 +1,14 @@
import { useRef } from "react"; import { useRef } from "react";
import clsx from "clsx"; import clsx from "clsx";
import { Combobox as HeadlessCombobox, ComboboxInput, ComboboxOption, ComboboxOptions } from "@headlessui/react"; import {
Combobox as HeadlessCombobox,
ComboboxInput,
ComboboxOption,
ComboboxOptions,
} from "@headlessui/react";
import { cva } from "@/cva.config"; import { cva } from "@/cva.config";
import Card from "./Card"; import Card from "./Card";
export interface ComboboxOption { export interface ComboboxOption {
@ -22,7 +29,7 @@ const comboboxVariants = cva({
type BaseProps = React.ComponentProps<typeof HeadlessCombobox>; type BaseProps = React.ComponentProps<typeof HeadlessCombobox>;
interface ComboboxProps extends Omit<BaseProps, 'displayValue'> { interface ComboboxProps extends Omit<BaseProps, "displayValue"> {
displayValue: (option: ComboboxOption) => string; displayValue: (option: ComboboxOption) => string;
onInputChange: (option: string) => void; onInputChange: (option: string) => void;
options: () => ComboboxOption[]; options: () => ComboboxOption[];
@ -48,10 +55,7 @@ export function Combobox({
const classes = comboboxVariants({ size }); const classes = comboboxVariants({ size });
return ( return (
<HeadlessCombobox <HeadlessCombobox onChange={onChange} {...otherProps}>
onChange={onChange}
{...otherProps}
>
{() => ( {() => (
<> <>
<Card className="w-auto !border border-solid !border-slate-800/30 shadow outline-0 dark:!border-slate-300/30"> <Card className="w-auto !border border-solid !border-slate-800/30 shadow outline-0 dark:!border-slate-300/30">
@ -73,30 +77,31 @@ export function Combobox({
"focus:outline-blue-600 focus:ring-2 focus:ring-blue-700 focus:ring-offset-2 dark:focus:outline-blue-500 dark:focus:ring-blue-500", "focus:outline-blue-600 focus:ring-2 focus:ring-blue-700 focus:ring-offset-2 dark:focus:outline-blue-500 dark:focus:ring-blue-500",
// Disabled // Disabled
disabled && "pointer-events-none select-none bg-slate-50 text-slate-500/80 dark:bg-slate-800 dark:text-slate-400/80 disabled:hover:bg-white dark:disabled:hover:bg-slate-800" disabled &&
"pointer-events-none select-none bg-slate-50 text-slate-500/80 disabled:hover:bg-white dark:bg-slate-800 dark:text-slate-400/80 dark:disabled:hover:bg-slate-800",
)} )}
placeholder={disabled ? disabledMessage : placeholder} placeholder={disabled ? disabledMessage : placeholder}
displayValue={displayValue} displayValue={displayValue}
onChange={(event) => onInputChange(event.target.value)} onChange={event => onInputChange(event.target.value)}
disabled={disabled} disabled={disabled}
/> />
</Card> </Card>
{options().length > 0 && ( {options().length > 0 && (
<ComboboxOptions className="absolute left-0 z-[100] mt-1 w-full max-h-60 overflow-auto rounded-md bg-white py-1 text-sm shadow-lg ring-1 ring-black/5 dark:bg-slate-800 dark:ring-slate-700 hide-scrollbar"> <ComboboxOptions className="hide-scrollbar absolute left-0 z-[100] mt-1 max-h-60 w-full overflow-auto rounded-md bg-white py-1 text-sm shadow-lg ring-1 ring-black/5 dark:bg-slate-800 dark:ring-slate-700">
{options().map((option) => ( {options().map(option => (
<ComboboxOption <ComboboxOption
key={option.value} key={option.value}
value={option} value={option}
className={clsx( className={clsx(
// General styling // General styling
"cursor-default select-none py-2 px-4", "cursor-default select-none px-4 py-2",
// Hover and active states // Hover and active states
"hover:bg-blue-50/80 ui-active:bg-blue-50/80 ui-active:text-blue-900", "hover:bg-blue-50/80 ui-active:bg-blue-50/80 ui-active:text-blue-900",
// Dark mode // Dark mode
"dark:text-slate-300 dark:hover:bg-slate-700 dark:ui-active:bg-slate-700 dark:ui-active:text-blue-200" "dark:text-slate-300 dark:hover:bg-slate-700 dark:ui-active:bg-slate-700 dark:ui-active:text-blue-200",
)} )}
> >
{option.label} {option.label}
@ -106,10 +111,8 @@ export function Combobox({
)} )}
{options().length === 0 && inputRef.current?.value && ( {options().length === 0 && inputRef.current?.value && (
<div className="absolute left-0 z-[100] mt-1 w-full rounded-md bg-white dark:bg-slate-800 py-2 px-4 text-sm shadow-lg ring-1 ring-black/5 dark:ring-slate-700"> <div className="absolute left-0 z-[100] mt-1 w-full rounded-md bg-white px-4 py-2 text-sm shadow-lg ring-1 ring-black/5 dark:bg-slate-800 dark:ring-slate-700">
<div className="text-slate-500 dark:text-slate-400"> <div className="text-slate-500 dark:text-slate-400">{emptyMessage}</div>
{emptyMessage}
</div>
</div> </div>
)} )}
</> </>

View File

@ -1,4 +1,9 @@
import { ExclamationTriangleIcon, CheckCircleIcon, InformationCircleIcon } from "@heroicons/react/24/outline"; import {
ExclamationTriangleIcon,
CheckCircleIcon,
InformationCircleIcon,
} from "@heroicons/react/24/outline";
import { cx } from "@/cva.config"; import { cx } from "@/cva.config";
import { Button } from "@/components/Button"; import { Button } from "@/components/Button";
import Modal from "@/components/Modal"; import Modal from "@/components/Modal";
@ -42,12 +47,15 @@ const variantConfig = {
iconBgClass: "bg-blue-100", iconBgClass: "bg-blue-100",
buttonTheme: "primary", buttonTheme: "primary",
}, },
} as Record<Variant, { } as Record<
Variant,
{
icon: React.ElementType; icon: React.ElementType;
iconClass: string; iconClass: string;
iconBgClass: string; iconBgClass: string;
buttonTheme: "danger" | "primary" | "blank" | "light" | "lightDanger"; buttonTheme: "danger" | "primary" | "blank" | "light" | "lightDanger";
}>; }
>;
export function ConfirmDialog({ export function ConfirmDialog({
open, open,
@ -65,13 +73,18 @@ export function ConfirmDialog({
return ( return (
<Modal open={open} onClose={onClose}> <Modal open={open} onClose={onClose}>
<div className="mx-auto max-w-xl px-4 transition-all duration-300 ease-in-out"> <div className="mx-auto max-w-xl px-4 transition-all duration-300 ease-in-out">
<div className="relative w-full overflow-hidden rounded-lg bg-white p-6 text-left align-middle shadow-xl transition-all dark:bg-slate-800 pointer-events-auto"> <div className="pointer-events-auto relative w-full overflow-hidden rounded-lg bg-white p-6 text-left align-middle shadow-xl transition-all dark:bg-slate-800">
<div className="space-y-4"> <div className="space-y-4">
<div className="sm:flex sm:items-start"> <div className="sm:flex sm:items-start">
<div className={cx("mx-auto flex size-12 shrink-0 items-center justify-center rounded-full sm:mx-0 sm:size-10", iconBgClass)}> <div
className={cx(
"mx-auto flex size-12 shrink-0 items-center justify-center rounded-full sm:mx-0 sm:size-10",
iconBgClass,
)}
>
<Icon aria-hidden="true" className={cx("size-6", iconClass)} /> <Icon aria-hidden="true" className={cx("size-6", iconClass)} />
</div> </div>
<div className="mt-3 text-center sm:mt-0 sm:ml-4 sm:text-left"> <div className="mt-3 text-center sm:ml-4 sm:mt-0 sm:text-left">
<h2 className="text-lg font-bold leading-tight text-black dark:text-white"> <h2 className="text-lg font-bold leading-tight text-black dark:text-white">
{title} {title}
</h2> </h2>
@ -83,12 +96,7 @@ export function ConfirmDialog({
<div className="flex justify-end gap-x-2"> <div className="flex justify-end gap-x-2">
{cancelText && ( {cancelText && (
<Button <Button size="SM" theme="blank" text={cancelText} onClick={onClose} />
size="SM"
theme="blank"
text={cancelText}
onClick={onClose}
/>
)} )}
<Button <Button
size="SM" size="SM"

View File

@ -1,5 +1,4 @@
import { useState } from "react"; import { useState } from "react";
import { LuPlus } from "react-icons/lu"; import { LuPlus } from "react-icons/lu";
import { KeySequence } from "@/hooks/stores"; import { KeySequence } from "@/hooks/stores";
@ -7,16 +6,23 @@ import { Button } from "@/components/Button";
import { InputFieldWithLabel, FieldError } from "@/components/InputField"; import { InputFieldWithLabel, FieldError } from "@/components/InputField";
import Fieldset from "@/components/Fieldset"; import Fieldset from "@/components/Fieldset";
import { MacroStepCard } from "@/components/MacroStepCard"; import { MacroStepCard } from "@/components/MacroStepCard";
import { DEFAULT_DELAY, MAX_STEPS_PER_MACRO, MAX_KEYS_PER_STEP } from "@/constants/macros"; import {
DEFAULT_DELAY,
MAX_STEPS_PER_MACRO,
MAX_KEYS_PER_STEP,
} from "@/constants/macros";
import FieldLabel from "@/components/FieldLabel"; import FieldLabel from "@/components/FieldLabel";
interface ValidationErrors { interface ValidationErrors {
name?: string; name?: string;
steps?: Record<number, { steps?: Record<
number,
{
keys?: string; keys?: string;
modifiers?: string; modifiers?: string;
delay?: string; delay?: string;
}>; }
>;
} }
interface MacroFormProps { interface MacroFormProps {
@ -57,12 +63,14 @@ export function MacroForm({
if (!macro.steps?.length) { if (!macro.steps?.length) {
newErrors.steps = { 0: { keys: "At least one step is required" } }; newErrors.steps = { 0: { keys: "At least one step is required" } };
} else { } else {
const hasKeyOrModifier = macro.steps.some(step => const hasKeyOrModifier = macro.steps.some(
(step.keys?.length || 0) > 0 || (step.modifiers?.length || 0) > 0 step => (step.keys?.length || 0) > 0 || (step.modifiers?.length || 0) > 0,
); );
if (!hasKeyOrModifier) { if (!hasKeyOrModifier) {
newErrors.steps = { 0: { keys: "At least one step must have keys or modifiers" } }; newErrors.steps = {
0: { keys: "At least one step must have keys or modifiers" },
};
} }
} }
@ -87,7 +95,10 @@ export function MacroForm({
} }
}; };
const handleKeySelect = (stepIndex: number, option: { value: string | null; keys?: string[] }) => { const handleKeySelect = (
stepIndex: number,
option: { value: string | null; keys?: string[] },
) => {
const newSteps = [...(macro.steps || [])]; const newSteps = [...(macro.steps || [])];
if (!newSteps[stepIndex]) return; if (!newSteps[stepIndex]) return;
@ -97,7 +108,9 @@ export function MacroForm({
if (!newSteps[stepIndex].keys) { if (!newSteps[stepIndex].keys) {
newSteps[stepIndex].keys = []; newSteps[stepIndex].keys = [];
} }
const keysArray = Array.isArray(newSteps[stepIndex].keys) ? newSteps[stepIndex].keys : []; const keysArray = Array.isArray(newSteps[stepIndex].keys)
? newSteps[stepIndex].keys
: [];
if (keysArray.length >= MAX_KEYS_PER_STEP) { if (keysArray.length >= MAX_KEYS_PER_STEP) {
showTemporaryError(`Maximum of ${MAX_KEYS_PER_STEP} keys per step allowed`); showTemporaryError(`Maximum of ${MAX_KEYS_PER_STEP} keys per step allowed`);
return; return;
@ -148,9 +161,9 @@ export function MacroForm({
setMacro({ ...macro, steps: newSteps }); setMacro({ ...macro, steps: newSteps });
}; };
const handleStepMove = (stepIndex: number, direction: 'up' | 'down') => { const handleStepMove = (stepIndex: number, direction: "up" | "down") => {
const newSteps = [...(macro.steps || [])]; const newSteps = [...(macro.steps || [])];
const newIndex = direction === 'up' ? stepIndex - 1 : stepIndex + 1; const newIndex = direction === "up" ? stepIndex - 1 : stepIndex + 1;
[newSteps[stepIndex], newSteps[newIndex]] = [newSteps[newIndex], newSteps[stepIndex]]; [newSteps[stepIndex], newSteps[newIndex]] = [newSteps[newIndex], newSteps[stepIndex]];
setMacro({ ...macro, steps: newSteps }); setMacro({ ...macro, steps: newSteps });
}; };
@ -181,7 +194,10 @@ export function MacroForm({
<div> <div>
<div className="flex items-center justify-between text-sm"> <div className="flex items-center justify-between text-sm">
<div className="flex items-center gap-1"> <div className="flex items-center gap-1">
<FieldLabel label="Steps" description={`Keys/modifiers executed in sequence with a delay between each step.`} /> <FieldLabel
label="Steps"
description={`Keys/modifiers executed in sequence with a delay between each step.`}
/>
</div> </div>
<span className="text-slate-500 dark:text-slate-400"> <span className="text-slate-500 dark:text-slate-400">
{macro.steps?.length || 0}/{MAX_STEPS_PER_MACRO} steps {macro.steps?.length || 0}/{MAX_STEPS_PER_MACRO} steps
@ -199,18 +215,24 @@ export function MacroForm({
key={stepIndex} key={stepIndex}
step={step} step={step}
stepIndex={stepIndex} stepIndex={stepIndex}
onDelete={macro.steps && macro.steps.length > 1 ? () => { onDelete={
macro.steps && macro.steps.length > 1
? () => {
const newSteps = [...(macro.steps || [])]; const newSteps = [...(macro.steps || [])];
newSteps.splice(stepIndex, 1); newSteps.splice(stepIndex, 1);
setMacro(prev => ({ ...prev, steps: newSteps })); setMacro(prev => ({ ...prev, steps: newSteps }));
} : undefined} }
onMoveUp={() => handleStepMove(stepIndex, 'up')} : undefined
onMoveDown={() => handleStepMove(stepIndex, 'down')} }
onKeySelect={(option) => handleKeySelect(stepIndex, option)} onMoveUp={() => handleStepMove(stepIndex, "up")}
onKeyQueryChange={(query) => handleKeyQueryChange(stepIndex, query)} onMoveDown={() => handleStepMove(stepIndex, "down")}
keyQuery={keyQueries[stepIndex] || ''} onKeySelect={option => handleKeySelect(stepIndex, option)}
onModifierChange={(modifiers) => handleModifierChange(stepIndex, modifiers)} onKeyQueryChange={query => handleKeyQueryChange(stepIndex, query)}
onDelayChange={(delay) => handleDelayChange(stepIndex, delay)} keyQuery={keyQueries[stepIndex] || ""}
onModifierChange={modifiers =>
handleModifierChange(stepIndex, modifiers)
}
onDelayChange={delay => handleDelayChange(stepIndex, delay)}
isLastStep={stepIndex === (macro.steps?.length || 0) - 1} isLastStep={stepIndex === (macro.steps?.length || 0) - 1}
/> />
))} ))}
@ -223,10 +245,12 @@ export function MacroForm({
theme="light" theme="light"
fullWidth fullWidth
LeadingIcon={LuPlus} LeadingIcon={LuPlus}
text={`Add Step ${isMaxStepsReached ? `(${MAX_STEPS_PER_MACRO} max)` : ''}`} text={`Add Step ${isMaxStepsReached ? `(${MAX_STEPS_PER_MACRO} max)` : ""}`}
onClick={() => { onClick={() => {
if (isMaxStepsReached) { if (isMaxStepsReached) {
showTemporaryError(`You can only add a maximum of ${MAX_STEPS_PER_MACRO} steps per macro.`); showTemporaryError(
`You can only add a maximum of ${MAX_STEPS_PER_MACRO} steps per macro.`,
);
return; return;
} }
@ -234,7 +258,7 @@ export function MacroForm({
...prev, ...prev,
steps: [ steps: [
...(prev.steps || []), ...(prev.steps || []),
{ keys: [], modifiers: [], delay: DEFAULT_DELAY } { keys: [], modifiers: [], delay: DEFAULT_DELAY },
], ],
})); }));
setErrors({}); setErrors({});
@ -257,12 +281,7 @@ export function MacroForm({
onClick={handleSubmit} onClick={handleSubmit}
disabled={isSubmitting} disabled={isSubmitting}
/> />
<Button <Button size="SM" theme="light" text="Cancel" onClick={onCancel} />
size="SM"
theme="light"
text="Cancel"
onClick={onCancel}
/>
</div> </div>
</div> </div>
</div> </div>

View File

@ -3,11 +3,11 @@ import { ExclamationTriangleIcon } from "@heroicons/react/24/solid";
import { ArrowPathIcon, ArrowRightIcon } from "@heroicons/react/16/solid"; import { ArrowPathIcon, ArrowRightIcon } from "@heroicons/react/16/solid";
import { motion, AnimatePresence } from "framer-motion"; import { motion, AnimatePresence } from "framer-motion";
import { LuPlay } from "react-icons/lu"; import { LuPlay } from "react-icons/lu";
import { BsMouseFill } from "react-icons/bs";
import { Button, LinkButton } from "@components/Button"; import { Button, LinkButton } from "@components/Button";
import LoadingSpinner from "@components/LoadingSpinner"; import LoadingSpinner from "@components/LoadingSpinner";
import Card, { GridCard } from "@components/Card"; import Card, { GridCard } from "@components/Card";
import { BsMouseFill } from "react-icons/bs";
interface OverlayContentProps { interface OverlayContentProps {
children: React.ReactNode; children: React.ReactNode;
@ -242,8 +242,8 @@ export function HDMIErrorOverlay({ show, hdmiState }: HDMIErrorOverlayProps) {
Ensure source device is powered on and outputting a signal Ensure source device is powered on and outputting a signal
</li> </li>
<li> <li>
If using an adapter, ensure it&apos;s compatible and If using an adapter, ensure it&apos;s compatible and functioning
functioning correctly correctly
</li> </li>
</ul> </ul>
</div> </div>

View File

@ -151,7 +151,7 @@ export default function WebRTCVideo() {
const isKeyboardLockGranted = await checkNavigatorPermissions("keyboard-lock"); const isKeyboardLockGranted = await checkNavigatorPermissions("keyboard-lock");
if (isKeyboardLockGranted) { if (isKeyboardLockGranted) {
if ("keyboard" in navigator) { if ("keyboard" in navigator) {
// @ts-ignore // @ts-expect-error - keyboard lock is not supported in all browsers
await navigator.keyboard.lock(); await navigator.keyboard.lock();
} }
} }

View File

@ -1,6 +1,11 @@
import { create } from "zustand"; import { create } from "zustand";
import { createJSONStorage, persist } from "zustand/middleware"; import { createJSONStorage, persist } from "zustand/middleware";
import { MAX_STEPS_PER_MACRO, MAX_TOTAL_MACROS, MAX_KEYS_PER_STEP } from "@/constants/macros";
import {
MAX_STEPS_PER_MACRO,
MAX_TOTAL_MACROS,
MAX_KEYS_PER_STEP,
} from "@/constants/macros";
// Define the JsonRpc types for better type checking // Define the JsonRpc types for better type checking
interface JsonRpcResponse { interface JsonRpcResponse {
@ -719,12 +724,23 @@ export interface NetworkState {
setDhcpLeaseExpiry: (expiry: Date) => void; setDhcpLeaseExpiry: (expiry: Date) => void;
} }
export type IPv6Mode =
export type IPv6Mode = "disabled" | "slaac" | "dhcpv6" | "slaac_and_dhcpv6" | "static" | "link_local" | "unknown"; | "disabled"
| "slaac"
| "dhcpv6"
| "slaac_and_dhcpv6"
| "static"
| "link_local"
| "unknown";
export type IPv4Mode = "disabled" | "static" | "dhcp" | "unknown"; export type IPv4Mode = "disabled" | "static" | "dhcp" | "unknown";
export type LLDPMode = "disabled" | "basic" | "all" | "unknown"; export type LLDPMode = "disabled" | "basic" | "all" | "unknown";
export type mDNSMode = "disabled" | "auto" | "ipv4_only" | "ipv6_only" | "unknown"; export type mDNSMode = "disabled" | "auto" | "ipv4_only" | "ipv6_only" | "unknown";
export type TimeSyncMode = "ntp_only" | "ntp_and_http" | "http_only" | "custom" | "unknown"; export type TimeSyncMode =
| "ntp_only"
| "ntp_and_http"
| "http_only"
| "custom"
| "unknown";
export interface NetworkSettings { export interface NetworkSettings {
hostname: string; hostname: string;
@ -749,7 +765,7 @@ export const useNetworkStateStore = create<NetworkState>((set, get) => ({
lease.lease_expiry = expiry; lease.lease_expiry = expiry;
set({ dhcp_lease: lease }); set({ dhcp_lease: lease });
} },
})); }));
export interface KeySequenceStep { export interface KeySequenceStep {
@ -771,8 +787,20 @@ export interface MacrosState {
initialized: boolean; initialized: boolean;
loadMacros: () => Promise<void>; loadMacros: () => Promise<void>;
saveMacros: (macros: KeySequence[]) => Promise<void>; saveMacros: (macros: KeySequence[]) => Promise<void>;
sendFn: ((method: string, params: unknown, callback?: ((resp: JsonRpcResponse) => void) | undefined) => void) | null; sendFn:
setSendFn: (sendFn: ((method: string, params: unknown, callback?: ((resp: JsonRpcResponse) => void) | undefined) => void)) => void; | ((
method: string,
params: unknown,
callback?: ((resp: JsonRpcResponse) => void) | undefined,
) => void)
| null;
setSendFn: (
sendFn: (
method: string,
params: unknown,
callback?: ((resp: JsonRpcResponse) => void) | undefined,
) => void,
) => void;
} }
export const generateMacroId = () => { export const generateMacroId = () => {
@ -785,7 +813,7 @@ export const useMacrosStore = create<MacrosState>((set, get) => ({
initialized: false, initialized: false,
sendFn: null, sendFn: null,
setSendFn: (sendFn) => { setSendFn: sendFn => {
set({ sendFn }); set({ sendFn });
}, },
@ -802,7 +830,7 @@ export const useMacrosStore = create<MacrosState>((set, get) => ({
try { try {
await new Promise<void>((resolve, reject) => { await new Promise<void>((resolve, reject) => {
sendFn("getKeyboardMacros", {}, (response) => { sendFn("getKeyboardMacros", {}, response => {
if (response.error) { if (response.error) {
console.error("Error loading macros:", response.error); console.error("Error loading macros:", response.error);
reject(new Error(response.error.message)); reject(new Error(response.error.message));
@ -822,7 +850,7 @@ export const useMacrosStore = create<MacrosState>((set, get) => ({
set({ set({
macros: sortedMacros, macros: sortedMacros,
initialized: true initialized: true,
}); });
resolve(); resolve();
@ -849,15 +877,23 @@ export const useMacrosStore = create<MacrosState>((set, get) => ({
for (const macro of macros) { for (const macro of macros) {
if (macro.steps.length > MAX_STEPS_PER_MACRO) { if (macro.steps.length > MAX_STEPS_PER_MACRO) {
console.error(`Cannot save: macro "${macro.name}" exceeds maximum of ${MAX_STEPS_PER_MACRO} steps`); console.error(
throw new Error(`Cannot save: macro "${macro.name}" exceeds maximum of ${MAX_STEPS_PER_MACRO} steps`); `Cannot save: macro "${macro.name}" exceeds maximum of ${MAX_STEPS_PER_MACRO} steps`,
);
throw new Error(
`Cannot save: macro "${macro.name}" exceeds maximum of ${MAX_STEPS_PER_MACRO} steps`,
);
} }
for (let i = 0; i < macro.steps.length; i++) { for (let i = 0; i < macro.steps.length; i++) {
const step = macro.steps[i]; const step = macro.steps[i];
if (step.keys && step.keys.length > MAX_KEYS_PER_STEP) { if (step.keys && step.keys.length > MAX_KEYS_PER_STEP) {
console.error(`Cannot save: macro "${macro.name}" step ${i + 1} exceeds maximum of ${MAX_KEYS_PER_STEP} keys`); console.error(
throw new Error(`Cannot save: macro "${macro.name}" step ${i + 1} exceeds maximum of ${MAX_KEYS_PER_STEP} keys`); `Cannot save: macro "${macro.name}" step ${i + 1} exceeds maximum of ${MAX_KEYS_PER_STEP} keys`,
);
throw new Error(
`Cannot save: macro "${macro.name}" step ${i + 1} exceeds maximum of ${MAX_KEYS_PER_STEP} keys`,
);
} }
} }
} }
@ -867,18 +903,23 @@ export const useMacrosStore = create<MacrosState>((set, get) => ({
try { try {
const macrosWithSortOrder = macros.map((macro, index) => ({ const macrosWithSortOrder = macros.map((macro, index) => ({
...macro, ...macro,
sortOrder: macro.sortOrder !== undefined ? macro.sortOrder : index sortOrder: macro.sortOrder !== undefined ? macro.sortOrder : index,
})); }));
const response = await new Promise<JsonRpcResponse>((resolve) => { const response = await new Promise<JsonRpcResponse>(resolve => {
sendFn("setKeyboardMacros", { params: { macros: macrosWithSortOrder } }, (response) => { sendFn(
"setKeyboardMacros",
{ params: { macros: macrosWithSortOrder } },
response => {
resolve(response); resolve(response);
}); },
);
}); });
if (response.error) { if (response.error) {
console.error("Error saving macros:", response.error); console.error("Error saving macros:", response.error);
const errorMessage = typeof response.error.data === 'string' const errorMessage =
typeof response.error.data === "string"
? response.error.data ? response.error.data
: response.error.message || "Failed to save macros"; : response.error.message || "Failed to save macros";
throw new Error(errorMessage); throw new Error(errorMessage);
@ -892,5 +933,5 @@ export const useMacrosStore = create<MacrosState>((set, get) => ({
} finally { } finally {
set({ loading: false }); set({ loading: false });
} }
} },
})); }));

View File

@ -1,6 +1,15 @@
import { useEffect, Fragment, useMemo, useState, useCallback } from "react"; import { useEffect, Fragment, useMemo, useState, useCallback } from "react";
import { useNavigate } from "react-router-dom"; import { useNavigate } from "react-router-dom";
import { LuPenLine, LuCopy, LuMoveRight, LuCornerDownRight, LuArrowUp, LuArrowDown, LuTrash2, LuCommand } from "react-icons/lu"; import {
LuPenLine,
LuCopy,
LuMoveRight,
LuCornerDownRight,
LuArrowUp,
LuArrowDown,
LuTrash2,
LuCommand,
} from "react-icons/lu";
import { KeySequence, useMacrosStore, generateMacroId } from "@/hooks/stores"; import { KeySequence, useMacrosStore, generateMacroId } from "@/hooks/stores";
import { SettingsPageHeader } from "@/components/SettingsPageheader"; import { SettingsPageHeader } from "@/components/SettingsPageheader";
@ -27,9 +36,9 @@ export default function SettingsMacrosRoute() {
const [showDeleteConfirm, setShowDeleteConfirm] = useState(false); const [showDeleteConfirm, setShowDeleteConfirm] = useState(false);
const [macroToDelete, setMacroToDelete] = useState<KeySequence | null>(null); const [macroToDelete, setMacroToDelete] = useState<KeySequence | null>(null);
const isMaxMacrosReached = useMemo(() => const isMaxMacrosReached = useMemo(
macros.length >= MAX_TOTAL_MACROS, () => macros.length >= MAX_TOTAL_MACROS,
[macros.length] [macros.length],
); );
useEffect(() => { useEffect(() => {
@ -38,7 +47,8 @@ export default function SettingsMacrosRoute() {
} }
}, [initialized, loadMacros]); }, [initialized, loadMacros]);
const handleDuplicateMacro = useCallback(async (macro: KeySequence) => { const handleDuplicateMacro = useCallback(
async (macro: KeySequence) => {
if (!macro?.id || !macro?.name) { if (!macro?.id || !macro?.name) {
notifications.error("Invalid macro data"); notifications.error("Invalid macro data");
return; return;
@ -70,15 +80,18 @@ export default function SettingsMacrosRoute() {
} finally { } finally {
setActionLoadingId(null); setActionLoadingId(null);
} }
}, [isMaxMacrosReached, macros, saveMacros, setActionLoadingId]); },
[isMaxMacrosReached, macros, saveMacros, setActionLoadingId],
);
const handleMoveMacro = useCallback(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) { if (!Array.isArray(macros) || macros.length === 0) {
notifications.error("No macros available"); notifications.error("No macros available");
return; return;
} }
const newIndex = direction === 'up' ? index - 1 : index + 1; const newIndex = direction === "up" ? index - 1 : index + 1;
if (newIndex < 0 || newIndex >= macros.length) return; if (newIndex < 0 || newIndex >= macros.length) return;
setActionLoadingId(macroId); setActionLoadingId(macroId);
@ -99,14 +112,18 @@ export default function SettingsMacrosRoute() {
} finally { } finally {
setActionLoadingId(null); setActionLoadingId(null);
} }
}, [macros, saveMacros, setActionLoadingId]); },
[macros, saveMacros, setActionLoadingId],
);
const handleDeleteMacro = useCallback(async () => { const handleDeleteMacro = useCallback(async () => {
if (!macroToDelete?.id) return; if (!macroToDelete?.id) return;
setActionLoadingId(macroToDelete.id); setActionLoadingId(macroToDelete.id);
try { try {
const updatedMacros = normalizeSortOrders(macros.filter(m => m.id !== macroToDelete.id)); const updatedMacros = normalizeSortOrders(
macros.filter(m => m.id !== macroToDelete.id),
);
await saveMacros(updatedMacros); await saveMacros(updatedMacros);
notifications.success(`Macro "${macroToDelete.name}" deleted successfully`); notifications.success(`Macro "${macroToDelete.name}" deleted successfully`);
setShowDeleteConfirm(false); setShowDeleteConfirm(false);
@ -122,16 +139,17 @@ export default function SettingsMacrosRoute() {
} }
}, [macroToDelete, macros, saveMacros]); }, [macroToDelete, macros, saveMacros]);
const MacroList = useMemo(() => ( const MacroList = useMemo(
() => (
<div className="space-y-2"> <div className="space-y-2">
{macros.map((macro, index) => ( {macros.map((macro, index) => (
<Card key={macro.id} className="p-2 bg-white dark:bg-slate-800"> <Card key={macro.id} className="bg-white p-2 dark:bg-slate-800">
<div className="flex items-center justify-between"> <div className="flex items-center justify-between">
<div className="flex flex-col gap-1 px-2"> <div className="flex flex-col gap-1 px-2">
<Button <Button
size="XS" size="XS"
theme="light" theme="light"
onClick={() => handleMoveMacro(index, 'up', macro.id)} onClick={() => handleMoveMacro(index, "up", macro.id)}
disabled={index === 0 || actionLoadingId === macro.id} disabled={index === 0 || actionLoadingId === macro.id}
LeadingIcon={LuArrowUp} LeadingIcon={LuArrowUp}
aria-label={`Move ${macro.name} up`} aria-label={`Move ${macro.name} up`}
@ -139,59 +157,79 @@ export default function SettingsMacrosRoute() {
<Button <Button
size="XS" size="XS"
theme="light" theme="light"
onClick={() => handleMoveMacro(index, 'down', macro.id)} onClick={() => handleMoveMacro(index, "down", macro.id)}
disabled={index === macros.length - 1 || actionLoadingId === macro.id} disabled={index === macros.length - 1 || actionLoadingId === macro.id}
LeadingIcon={LuArrowDown} LeadingIcon={LuArrowDown}
aria-label={`Move ${macro.name} down`} aria-label={`Move ${macro.name} down`}
/> />
</div> </div>
<div className="flex-1 min-w-0 flex flex-col justify-center ml-2"> <div className="ml-2 flex min-w-0 flex-1 flex-col justify-center">
<h3 className="truncate text-sm font-semibold text-black dark:text-white"> <h3 className="truncate text-sm font-semibold text-black dark:text-white">
{macro.name} {macro.name}
</h3> </h3>
<p className="mt-1 ml-4 text-xs text-slate-500 dark:text-slate-400 overflow-hidden"> <p className="ml-4 mt-1 overflow-hidden text-xs text-slate-500 dark:text-slate-400">
<span className="flex flex-col items-start gap-1"> <span className="flex flex-col items-start gap-1">
{macro.steps.map((step, stepIndex) => { {macro.steps.map((step, stepIndex) => {
const StepIcon = stepIndex === 0 ? LuMoveRight : LuCornerDownRight; const StepIcon = stepIndex === 0 ? LuMoveRight : LuCornerDownRight;
return ( return (
<span key={stepIndex} className="inline-flex items-center"> <span key={stepIndex} className="inline-flex items-center">
<StepIcon className="mr-1 text-slate-400 dark:text-slate-500 h-3 w-3 flex-shrink-0" /> <StepIcon className="mr-1 h-3 w-3 flex-shrink-0 text-slate-400 dark:text-slate-500" />
<span className="bg-slate-50 dark:bg-slate-800 px-2 py-0.5 rounded-md border border-slate-200/50 dark:border-slate-700/50"> <span className="rounded-md border border-slate-200/50 bg-slate-50 px-2 py-0.5 dark:border-slate-700/50 dark:bg-slate-800">
{(Array.isArray(step.modifiers) && step.modifiers.length > 0) || (Array.isArray(step.keys) && step.keys.length > 0) ? ( {(Array.isArray(step.modifiers) &&
step.modifiers.length > 0) ||
(Array.isArray(step.keys) && step.keys.length > 0) ? (
<> <>
{Array.isArray(step.modifiers) && step.modifiers.map((modifier, idx) => ( {Array.isArray(step.modifiers) &&
step.modifiers.map((modifier, idx) => (
<Fragment key={`mod-${idx}`}> <Fragment key={`mod-${idx}`}>
<span className="font-medium text-slate-600 dark:text-slate-200"> <span className="font-medium text-slate-600 dark:text-slate-200">
{modifierDisplayMap[modifier] || modifier} {modifierDisplayMap[modifier] || modifier}
</span> </span>
{idx < step.modifiers.length - 1 && ( {idx < step.modifiers.length - 1 && (
<span className="text-slate-400 dark:text-slate-600"> + </span> <span className="text-slate-400 dark:text-slate-600">
{" "}
+{" "}
</span>
)} )}
</Fragment> </Fragment>
))} ))}
{Array.isArray(step.modifiers) && step.modifiers.length > 0 && Array.isArray(step.keys) && step.keys.length > 0 && ( {Array.isArray(step.modifiers) &&
<span className="text-slate-400 dark:text-slate-600"> + </span> step.modifiers.length > 0 &&
Array.isArray(step.keys) &&
step.keys.length > 0 && (
<span className="text-slate-400 dark:text-slate-600">
{" "}
+{" "}
</span>
)} )}
{Array.isArray(step.keys) && step.keys.map((key, idx) => ( {Array.isArray(step.keys) &&
step.keys.map((key, idx) => (
<Fragment key={`key-${idx}`}> <Fragment key={`key-${idx}`}>
<span className="font-medium text-blue-600 dark:text-blue-400"> <span className="font-medium text-blue-600 dark:text-blue-400">
{keyDisplayMap[key] || key} {keyDisplayMap[key] || key}
</span> </span>
{idx < step.keys.length - 1 && ( {idx < step.keys.length - 1 && (
<span className="text-slate-400 dark:text-slate-600"> + </span> <span className="text-slate-400 dark:text-slate-600">
{" "}
+{" "}
</span>
)} )}
</Fragment> </Fragment>
))} ))}
</> </>
) : ( ) : (
<span className="font-medium text-slate-500 dark:text-slate-400">Delay only</span> <span className="font-medium text-slate-500 dark:text-slate-400">
Delay only
</span>
)} )}
{step.delay !== DEFAULT_DELAY && ( {step.delay !== DEFAULT_DELAY && (
<span className="ml-1 text-slate-400 dark:text-slate-500">({step.delay}ms)</span> <span className="ml-1 text-slate-400 dark:text-slate-500">
({step.delay}ms)
</span>
)} )}
</span> </span>
</span> </span>
@ -201,7 +239,7 @@ export default function SettingsMacrosRoute() {
</p> </p>
</div> </div>
<div className="flex items-center gap-1 ml-4"> <div className="ml-4 flex items-center gap-1">
<Button <Button
size="XS" size="XS"
className="text-red-500 dark:text-red-400" className="text-red-500 dark:text-red-400"
@ -250,7 +288,19 @@ export default function SettingsMacrosRoute() {
isConfirming={actionLoadingId === macroToDelete?.id} isConfirming={actionLoadingId === macroToDelete?.id}
/> />
</div> </div>
), [macros, actionLoadingId, showDeleteConfirm, macroToDelete, handleDeleteMacro]); ),
[
macros,
showDeleteConfirm,
macroToDelete?.name,
macroToDelete?.id,
actionLoadingId,
handleDeleteMacro,
handleMoveMacro,
handleDuplicateMacro,
navigate,
],
);
return ( return (
<div className="space-y-4"> <div className="space-y-4">
@ -299,7 +349,9 @@ export default function SettingsMacrosRoute() {
/> />
} }
/> />
) : MacroList} ) : (
MacroList
)}
</div> </div>
</div> </div>
); );

View File

@ -15,17 +15,17 @@ import {
} from "@/hooks/stores"; } from "@/hooks/stores";
import { useJsonRpc } from "@/hooks/useJsonRpc"; import { useJsonRpc } from "@/hooks/useJsonRpc";
import notifications from "@/notifications"; import notifications from "@/notifications";
import { Button, LinkButton } from "@components/Button"; import { Button } from "@components/Button";
import { GridCard } from "@components/Card"; import { GridCard } from "@components/Card";
import InputField from "@components/InputField"; import InputField from "@components/InputField";
import { SettingsPageHeader } from "../components/SettingsPageheader"; import { SettingsPageHeader } from "../components/SettingsPageheader";
import { SelectMenuBasic } from "../components/SelectMenuBasic"; import { SelectMenuBasic } from "../components/SelectMenuBasic";
import { SettingsItem } from "./devices.$id.settings";
import Fieldset from "../components/Fieldset"; import Fieldset from "../components/Fieldset";
import { ConfirmDialog } from "../components/ConfirmDialog"; import { ConfirmDialog } from "../components/ConfirmDialog";
import { SettingsItem } from "./devices.$id.settings";
dayjs.extend(relativeTime); dayjs.extend(relativeTime);
const defaultNetworkSettings: NetworkSettings = { const defaultNetworkSettings: NetworkSettings = {