Closes #3181 [0]. The goal of this series is to extend the information displayed in the confirmation dialogs when cloning, migrating, removing VMs or CTs by adding the guest name in addition to the VM ID. This also unifies the displayed information since there are multiple actions that already include the guest name, such as reboot and shutdown [1]. This makes it easier for users to tell which VM or CT they are currently targeting, reducing the likelihood of erroneous actions.
To make the information in the dialogs consistent with sorting behavior selected by the user, "VMID (VM name)" is displayed if no sorting field has been explicitly set or the sorting field "VMID" was set through the tree settings. If "Name" was selected, the information is displayed as "VM name (VMID)". This affects the following actions performed on VMs/CTs: * Remove * Clone * Migrate * Snapshot * Snapshot rollback * Backup VM/CT from config view * Restore VM/CT from config view [0] https://bugzilla.proxmox.com/show_bug.cgi?id=3181 [1] https://lore.proxmox.com/pve-devel/96d831f7-8435-412a-8205-b33f71357...@proxmox.com Changes since v2: * Rebase on current master * Use nullish coalescing operator in call to format_task_description in SafeDestroy window proxmox-widget-toolkit: Michael Köppl (1): close #3181: ui: add formatted guest identifier to safe destroy dialog window src/window/SafeDestroy.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) pve-manager: Michael Köppl (1): close #3181: ui: display guest name in confirm dialogs www/manager6/Utils.js | 13 ++++++++++++- www/manager6/grid/BackupView.js | 4 ++++ www/manager6/lxc/CmdMenu.js | 9 ++++++++- www/manager6/lxc/Config.js | 15 +++++++++++++-- www/manager6/qemu/CmdMenu.js | 9 ++++++++- www/manager6/qemu/Config.js | 15 +++++++++++++-- www/manager6/tree/SnapshotTree.js | 12 ++++++++++-- www/manager6/window/Backup.js | 6 +++--- www/manager6/window/Clone.js | 7 +++++-- www/manager6/window/Migrate.js | 10 ++++++++-- www/manager6/window/Restore.js | 3 ++- www/manager6/window/Snapshot.js | 7 ++++++- 12 files changed, 92 insertions(+), 18 deletions(-) Summary over all repositories: 13 files changed, 99 insertions(+), 19 deletions(-) -- Generated by git-murpp 0.8.0 _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
