upstream/nextcloud
1
0
Fork 0
mirror of https://github.com/nextcloud/server.git synced 2026-06-10 09:13:19 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 54

fix(dialogs): migrate buttons to variant prop

Browse source 
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 <max@nextcloud.com>
This commit is contained in:
Max 2025-10-18 20:12:27 +02:00 committed by Ferdinand Thiessen
parent 689cbdf790
commit 463a5bd033
1 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
core/src/OC/dialogs.js
View file

@ -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)