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

Daniel Kral d.kral at proxmox.com
Fri Sep 26 16:08:00 CEST 2025


On Wed Sep 24, 2025 at 6:17 PM CEST, Michael Köppl wrote:
> The ConfirmRemoveDialog offers additional customization, but covers
> SafeDestroy's feature set. This is done mostly to streamline the
> implementation, basing dialogs to remove guests, storage, or resources
> on the same component.
>
> Signed-off-by: Michael Köppl <m.koeppl at proxmox.com>

SafeDestroy is also used in the PBS' web interface, so it should be
adapted there too:

```
$ 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');
```




More information about the pve-devel mailing list