From 463a5bd0334c1faf4549dd3300fecb85d382aeb8 Mon Sep 17 00:00:00 2001 From: Max Date: Sat, 18 Oct 2025 20:12:27 +0200 Subject: [PATCH] fix(dialogs): migrate buttons to variant prop In Nextcloud Vue v8 the `type` prop for dialog buttons was deprecated and replaces with the `variant` prop. In v9 the deprecated props are removed. Use the `variant` prop in the deprecated `OC.dialogs`. This is a follow up for #55726. Signed-off-by: Max --- core/src/OC/dialogs.js | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/core/src/OC/dialogs.js b/core/src/OC/dialogs.js index 94df7e8c331..a2d491ba14d 100644 --- a/core/src/OC/dialogs.js +++ b/core/src/OC/dialogs.js @@ -110,7 +110,7 @@ const Dialogs = { ? [ { label: t('core', 'Yes'), - type: 'error', + variant: 'error', callback: () => { callback.clicked = true callback(true) @@ -148,7 +148,7 @@ const Dialogs = { }, { label: t('core', 'Yes'), - type: 'primary', + variant: 'primary', callback: () => { callback.clicked = true callback(true) @@ -271,7 +271,7 @@ const Dialogs = { builder.addButton({ callback: legacyCallback(callback, button.type), label: button.text, - type: button.defaultButton ? 'primary' : 'secondary', + variant: button.defaultButton ? 'primary' : 'secondary', }) }) } else { @@ -284,14 +284,14 @@ const Dialogs = { buttons.push({ callback: legacyCallback(callback, FilePickerType.Choose), label: node && !this.multiSelect ? t('core', 'Choose {file}', { file: target }) : t('core', 'Choose'), - type: 'primary', + variant: 'primary', }) } if (type === FilePickerType.CopyMove || type === FilePickerType.Copy) { buttons.push({ callback: legacyCallback(callback, FilePickerType.Copy), label: target ? t('core', 'Copy to {target}', { target }) : t('core', 'Copy'), - type: 'primary', + variant: 'primary', icon: IconCopy, }) } @@ -299,7 +299,7 @@ const Dialogs = { buttons.push({ callback: legacyCallback(callback, FilePickerType.Move), label: target ? t('core', 'Move to {target}', { target }) : t('core', 'Move'), - type: type === FilePickerType.Move ? 'primary' : 'secondary', + variant: type === FilePickerType.Move ? 'primary' : 'secondary', icon: IconMove, }) } @@ -374,7 +374,7 @@ const Dialogs = { }) buttonList.push({ label: buttons?.confirm ?? t('core', 'Yes'), - type: 'primary', + variant: 'primary', callback: () => { callback._clicked = true callback(true) @@ -384,7 +384,7 @@ const Dialogs = { case Dialogs.OK_BUTTONS: buttonList.push({ label: buttons?.confirm ?? t('core', 'OK'), - type: 'primary', + variant: 'primary', callback: () => { callback._clicked = true callback(true)