[pve-devel] [PATCH manager/widget-toolkit v2 0/2] close #3181: ui: display guest name in confirm dialogs

[Michael Köppl]

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-b33f71357d0f@proxmox.com

Changes since v1:
* Add utils helper function for formatting guest information
* Make order of VM/CT ID and name dependent on sorting field chosen by
  user

proxmox-widget-toolkit:

Michael Köppl (1):
  close #3181: ui: add formatted guest identifier to safe destroy dialog
    window

 src/window/SafeDestroy.js | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)


pve-manager:

Michael Köppl (1):
  close #3181: ui: display guest name in confirm dialogs

 www/manager6/Utils.js             | 10 +++++++++-
 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, 89 insertions(+), 18 deletions(-)


Summary over all repositories:
  13 files changed, 95 insertions(+), 19 deletions(-)

-- 
Generated by git-murpp 0.8.0