[pve-devel] [PATCH manager v7 10/14] ui: tree/ResourceTree: show Tags in tree

Aaron Lauterer a.lauterer at proxmox.com
Wed Sep 14 16:15:16 CEST 2022


Why the change from vm.text to vm_text in {lxc,qemu}/Config.js?

AFAICT we have exactly the same string in the now not used "vm.text".

If these changes are needed and should be part of this commmit, a short 
explanation would be good as it does not seem to have anything to do with the 
resource tree.

On 6/21/22 11:20, Dominik Csapak wrote:
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>   www/manager6/lxc/Config.js        | 4 +++-
>   www/manager6/qemu/Config.js       | 4 +++-
>   www/manager6/tree/ResourceTree.js | 4 ++++
>   3 files changed, 10 insertions(+), 2 deletions(-)
> 
> diff --git a/www/manager6/lxc/Config.js b/www/manager6/lxc/Config.js
> index 9f1994d3..1b79628e 100644
> --- a/www/manager6/lxc/Config.js
> +++ b/www/manager6/lxc/Config.js
> @@ -206,8 +206,10 @@ Ext.define('PVE.lxc.Config', {
>   	    },
>   	});
>   
> +	let vm_text = `${vm.vmid} (${vm.name})`;
> +
>   	Ext.apply(me, {
> -	    title: Ext.String.format(gettext("Container {0} on node '{1}'"), vm.text, nodename),
> +	    title: Ext.String.format(gettext("Container {0} on node '{1}'"), vm_text, nodename),
>   	    hstateid: 'lxctab',
>   	    tbarSpacing: false,
>   	    tbar: [statusTxt, tagsContainer, '->', startBtn, shutdownBtn, migrateBtn, consoleBtn, moreBtn],
> diff --git a/www/manager6/qemu/Config.js b/www/manager6/qemu/Config.js
> index 2cd6d856..5c8fa620 100644
> --- a/www/manager6/qemu/Config.js
> +++ b/www/manager6/qemu/Config.js
> @@ -242,8 +242,10 @@ Ext.define('PVE.qemu.Config', {
>   	    },
>   	});
>   
> +	let vm_text = `${vm.vmid} (${vm.name})`;
> +
>   	Ext.apply(me, {
> -	    title: Ext.String.format(gettext("Virtual Machine {0} on node '{1}'"), vm.text, nodename),
> +	    title: Ext.String.format(gettext("Virtual Machine {0} on node '{1}'"), vm_text, nodename),
>   	    hstateid: 'kvmtab',
>   	    tbarSpacing: false,
>   	    tbar: [statusTxt, tagsContainer, '->', resumeBtn, startBtn, shutdownBtn, migrateBtn, consoleBtn, moreBtn],
> diff --git a/www/manager6/tree/ResourceTree.js b/www/manager6/tree/ResourceTree.js
> index 139defab..d41721b9 100644
> --- a/www/manager6/tree/ResourceTree.js
> +++ b/www/manager6/tree/ResourceTree.js
> @@ -5,6 +5,8 @@ Ext.define('PVE.tree.ResourceTree', {
>       extend: 'Ext.tree.TreePanel',
>       alias: ['widget.pveResourceTree'],
>   
> +    userCls: 'proxmox-tags-circle',
> +
>       statics: {
>   	typeDefaults: {
>   	    node: {
> @@ -114,6 +116,8 @@ Ext.define('PVE.tree.ResourceTree', {
>   	    }
>   	}
>   
> +	info.text += PVE.Utils.renderTags(info.tags, PVE.Utils.tagOverrides);
> +
>   	info.text = status + info.text;
>       },
>   





More information about the pve-devel mailing list