fix: [button-group] fixed the bug that multiple clicks on the same button would trigger multiple events (#2590)

zzcr · web-flow · commit 60e5402ec3ee · 2024-12-03T11:09:08.000+08:00
diff --git a/packages/renderless/src/button-group/index.ts b/packages/renderless/src/button-group/index.ts
@@ -22,7 +22,7 @@ export const handleChange =
 export const handleClick =
   ({ emit, props, state }: Pick<IButtonGroupRenderlessParams, 'emit' | 'props' | 'state'>) =>
   (node: IButtonGroupNode): void => {
-    if (!state.disabled && !node.disabled) {
+    if (!state.disabled && !node.disabled && state.value !== node[props.valueField]) {
       state.value = node[props.valueField]
       emit('update:modelValue', state.value)
     }

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ export const handleChange =`
`22`	`22`	`export const handleClick =`
`23`	`23`	`({ emit, props, state }: Pick<IButtonGroupRenderlessParams, 'emit' \| 'props' \| 'state'>) =>`
`24`	`24`	`(node: IButtonGroupNode): void => {`
`25`		`- if (!state.disabled && !node.disabled) {`
	`25`	`+ if (!state.disabled && !node.disabled && state.value !== node[props.valueField]) {`
`26`	`26`	`state.value = node[props.valueField]`
`27`	`27`	`emit('update:modelValue', state.value)`
`28`	`28`	`}`