[pve-devel] [PATCH manager] fix #6815: ui: resource tree: update tag colors properly

[Dominik Csapak]

forgot to write, this should be backported to stable-8 too, should be
cleanly cherry-pickable

On 10/10/25 9:28 AM, Dominik Csapak wrote:
> In commit:
>   e19de81f (ui: resource tree: fix change detection)
> the change detection for resource tree nodes was fixed so that it did
> not update on every refresh anymore.
> 
> This now also means that when the ui options are loaded (which contains
> the tag color overrides) we now have to correctly update nodes which
> contains tags, otherwise the standard colors will be shown.
> 
> There is already an event handler for the 'loadedUiOptions', so simply
> adapt it to also update the tree nodes that contains tags.
> 
> Fixes: e19de81f (ui: resource tree: fix change detection)
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>   www/manager6/tree/ResourceTree.js | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/www/manager6/tree/ResourceTree.js b/www/manager6/tree/ResourceTree.js
> index 315976e0..e83ccfc8 100644
> --- a/www/manager6/tree/ResourceTree.js
> +++ b/www/manager6/tree/ResourceTree.js
> @@ -605,7 +605,7 @@ Ext.define('PVE.tree.ResourceTree', {
>           me.mon(Ext.GlobalEvents, 'loadedUiOptions', () => {
>               me.store.getRootNode().cascadeBy({
>                   before: function (node) {
> -                    if (node.data.groupbyid) {
> +                    if (node.data.groupbyid || node.data.tags) {
>                           node.beginEdit();
>                           let info = node.data;
>                           me.setIconCls(info);