[pve-devel] [PATCH pve-manager] requires manual entering VM id before enabling the delete Button for VMs

Dietmar Maurer dietmar at proxmox.com
Mon Mar 21 10:55:26 CET 2016


applied, thanks.


> +	var msg = Ext.String.format(gettext('Are you sure you want to remove VM {0}?
> This will permanently erase all VM data.'), me.vmid);

This is still a bit confusing, because you use "VM {0}" for containers.
I would prefer CT {0} for containers.

I have already started such cleanup with patch:

https://git.proxmox.com/?p=pve-manager.git;a=commitdiff;h=31dbe7d5d14a6d15b5eb2787ddfc561c41767615#patch24

where I have done things like:

-                   var msg = Ext.String.format(gettext("Do you really want to
suspend CT {0}?"), vmid);
+                   var msg = Ext.String.format(gettext("Do you really want to
suspend {0}?"), gettext('CT') + ' ' + vmid);

Please can you take care of this?




More information about the pve-devel mailing list