[pve-devel] [PATCH FOLLOW-UP manager 1/1] ui: replace SafeDestroy with ConfirmRemoveDialog

Michael Köppl m.koeppl at proxmox.com
Tue Sep 30 10:04:32 CEST 2025


On Fri Sep 26, 2025 at 4:08 PM CEST, Daniel Kral wrote:
> SafeDestroy is also used in the PBS' web interface, so it should be
> adapted there too:

Thanks, I missed that one. Will send a separate patch to the pbs mailing
list as well!

>
> ```
> $ rg "(Proxmox\.window\.SafeDestroy|proxmoxSafeDestroy)" -tjs .
> ./pve/proxmox-widget-toolkit/src/window/SafeDestroy.js
> 3:Ext.define('Proxmox.window.SafeDestroy', {
> 5:    alias: 'widget.proxmoxSafeDestroy',
>
> ./pve/pve-manager/www/manager6/ceph/Pool.js
> 528:                            Ext.create('Proxmox.window.SafeDestroy', {
>
> ./pve/pve-manager/www/manager6/storage/ImageView.js
> 63:                    var win = Ext.create('Proxmox.window.SafeDestroy', {
>
> ./pve/pve-manager/www/manager6/window/SafeDestroyGuest.js
> 5:    extend: 'Proxmox.window.SafeDestroy',
>
> ./pve/pve-manager/www/manager6/window/SafeDestroyStorage.js
> 5:    extend: 'Proxmox.window.SafeDestroy',
>
> ./rust/proxmox-backup/www/DirectoryList.js
> 43:            Ext.create('Proxmox.window.SafeDestroy', {
>
> ./rust/proxmox-backup/www/datastore/Content.js
> 650:            Ext.create('Proxmox.window.SafeDestroy', {
>
> ./rust/proxmox-backup/www/datastore/OptionView.js
> 2:    extend: 'Proxmox.window.SafeDestroy',
>
> ./rust/proxmox-backup/www/window/NamespaceEdit.js
> 56:    extend: 'Proxmox.window.SafeDestroy',
> 77:                    let win = field.up('proxmoxSafeDestroy');
> ```
>
>
> _______________________________________________
> pve-devel mailing list
> pve-devel at lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel





More information about the pve-devel mailing list