Foundry/taf
1
0
Fork 0
Code Issues 5 Pull requests Releases 2 Wiki Activity Actions

Prevent the valueAttribute from being assigned to non-input types and clean up the branching

Browse source
This commit is contained in:
Oliver 2025-11-28 19:54:14 -07:00
parent 17a4f9343c
commit 03b3b9fb0f
1 changed files with 3 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

11
module/utils/DialogManager.mjs
View file

@ -60,18 +60,13 @@ export async function ask(
i.id ??= foundry.utils.randomID(16); i.id ??= foundry.utils.randomID(16);
i.key ??= i.label; i.key ??= i.label;
switch (i.type) {
case `input`: {
i.inputType ??= `text`;
}
}
// Only ever allow one input to claim autofocus // Only ever allow one input to claim autofocus
i.autofocus &&= !autofocusClaimed; i.autofocus &&= !autofocusClaimed;
autofocusClaimed ||= i.autofocus; autofocusClaimed ||= i.autofocus;
// Set the value's attribute name if it isn't specified explicitly // Set the value's attribute name if it isn't specified explicitly
if (!i.valueAttribute) { if (i.type === `input`) {
i.inputType ??= `text`;
switch (i.inputType) { switch (i.inputType) {
case `checkbox`: case `checkbox`:
i.type = `checkbox`; i.type = `checkbox`;
@ -79,7 +74,7 @@ export async function ask(
delete i.inputType; delete i.inputType;
break; break;
default: default:
i.valueAttribute = `value`; i.valueAttribute ??= `value`;
}; };
}; };
}; };