Merge pull request #15136 from Budibase/fix-multi-checkbox-value-coercion

Stop coercing all values to strings when using CoreCheckboxGroup
This commit is contained in:
Andrew Kingston 2024-12-10 14:58:48 +00:00 committed by GitHub
commit a3c1bbc12e
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 3 additions and 5 deletions

View File

@ -13,9 +13,8 @@
const dispatch = createEventDispatcher()
const onChange = e => {
const optionValue = e.target.value
if (e.target.checked && !value.includes(optionValue)) {
const onChange = optionValue => {
if (!value.includes(optionValue)) {
dispatch("change", [...value, optionValue])
} else {
dispatch(
@ -39,10 +38,9 @@
class="spectrum-Checkbox spectrum-Checkbox--sizeM spectrum-FieldGroup-item"
>
<input
on:change={onChange}
on:change={() => onChange(optionValue)}
type="checkbox"
class="spectrum-Checkbox-input"
value={optionValue}
checked={value.includes(optionValue)}
{disabled}
/>