Am 24.09.25 um 18:17 schrieb Michael Köppl: > The ConfirmRemoveDialog window covers the SafeDestroy window's > functionality, but makes it more general-purpose/flexible. It can be > used for both yes/no dialog as well as the SafeDestroy dialogs already > used for guests/storages. The reason for adding ConfirmRemoveDialog was > that Ext.Msg.MessageBox is not properly extendable and simply adding an > additional dialog would result in duplicate functionality covered in > both SafeDestroy and ConfirmRemoveDialog. Thus, replace SafeDestroy with > ConfirmRemoveDialog.
Why not keep SafeDestroy but make it base off the new ConfirmRemoveDialog so that it's basically just a very thin wrapper that does not cost us much to have? It's frequently used after all, so going that route would reduce churn quite a bit. _______________________________________________ pve-devel mailing list [email protected] https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
