Add a lower limit to duplicate tiles. (#2854)

* Add a limit to duplicate tiles.

* missed a change
This commit is contained in:
Will Hunt
2024-12-02 17:15:33 +00:00
committed by GitHub
parent a37c3bfb48
commit cc260811e2
2 changed files with 7 additions and 0 deletions

View File

@@ -73,6 +73,7 @@ interface InputFieldProps {
defaultValue?: string; defaultValue?: string;
placeholder?: string; placeholder?: string;
defaultChecked?: boolean; defaultChecked?: boolean;
min?: number;
onChange?: (event: ChangeEvent<HTMLInputElement>) => void; onChange?: (event: ChangeEvent<HTMLInputElement>) => void;
} }
@@ -91,6 +92,7 @@ export const InputField = forwardRef<
suffix, suffix,
description, description,
disabled, disabled,
min,
...rest ...rest
}, },
ref, ref,
@@ -127,6 +129,7 @@ export const InputField = forwardRef<
checked={checked} checked={checked}
disabled={disabled} disabled={disabled}
aria-describedby={descriptionId} aria-describedby={descriptionId}
min={min}
{...rest} {...rest}
/> />
)} )}

View File

@@ -227,9 +227,13 @@ export const SettingsModal: FC<Props> = ({
type="number" type="number"
label={t("developer_mode.duplicate_tiles_label")} label={t("developer_mode.duplicate_tiles_label")}
value={duplicateTiles.toString()} value={duplicateTiles.toString()}
min={0}
onChange={useCallback( onChange={useCallback(
(event: ChangeEvent<HTMLInputElement>): void => { (event: ChangeEvent<HTMLInputElement>): void => {
const value = event.target.valueAsNumber; const value = event.target.valueAsNumber;
if (value < 0) {
return;
}
setDuplicateTiles(Number.isNaN(value) ? 0 : value); setDuplicateTiles(Number.isNaN(value) ? 0 : value);
}, },
[setDuplicateTiles], [setDuplicateTiles],