From fd8d513deef7a929ec21ac3de0208870d84c8a8d Mon Sep 17 00:00:00 2001 From: Popescu Dan Date: Wed, 28 Dec 2022 19:30:13 +0200 Subject: [PATCH] fix(QRadio/QCheckbox): checked must be set as .prop to change the native form value and as .attr to change the html attr #15182 (#15183) close #15182 --- ui/src/components/checkbox/use-checkbox.js | 2 ++ ui/src/components/radio/QRadio.js | 2 ++ 2 files changed, 4 insertions(+) diff --git a/ui/src/components/checkbox/use-checkbox.js b/ui/src/components/checkbox/use-checkbox.js index 58cd12e68ab..77742672047 100644 --- a/ui/src/components/checkbox/use-checkbox.js +++ b/ui/src/components/checkbox/use-checkbox.js @@ -112,6 +112,8 @@ export default function (type, getInner) { const prop = { type: 'checkbox' } props.name !== void 0 && Object.assign(prop, { + // see https://vuejs.org/guide/extras/render-function.html#creating-vnodes (.prop) + '.checked': isTrue.value, '^checked': isTrue.value === true ? 'checked' : void 0, name: props.name, value: modelIsArray.value === true diff --git a/ui/src/components/radio/QRadio.js b/ui/src/components/radio/QRadio.js index 2eff88237e5..0cd46ba6888 100644 --- a/ui/src/components/radio/QRadio.js +++ b/ui/src/components/radio/QRadio.js @@ -100,6 +100,8 @@ export default createComponent({ const prop = { type: 'radio' } props.name !== void 0 && Object.assign(prop, { + // see https://vuejs.org/guide/extras/render-function.html#creating-vnodes (.prop) + '.checked': isTrue.value === true, '^checked': isTrue.value === true ? 'checked' : void 0, name: props.name, value: props.val