[PATCH manager 0/4] ui: small fixes for tag editing

[PATCH manager 0/4] ui: small fixes for tag editing

[Dominik Csapak]

Some small issues noticed when looking for the same issue as [0].

0: https://lore.proxmox.com/pve-devel/0c0f6a2b-b475-4f41-9342-c0d4ae94becc@inrae.fr/

Dominik Csapak (4):
  ui: form: list field: don't modify the class definition
  ui: form: list field: fix type for header text assignment
  ui: dc: options: add emptytext to user tag access edit grid
  ui: dc options: add missing field title to registered tag edit grid

 www/manager6/dc/RegisteredTagsEdit.js |  1 +
 www/manager6/dc/UserTagAccessEdit.js  |  1 +
 www/manager6/form/ListField.js        | 48 +++++++++++++--------------
 3 files changed, 26 insertions(+), 24 deletions(-)

-- 
2.47.3

[PATCH manager 1/4] ui: form: list field: don't modify the class definition

[Dominik Csapak]

Modifying a property defined on the class level modifies the prototype
of that class. This can have unintended consequences when only part of
the config is used in some widgets.

This can be seen here with the `Registered Tags` and `User Tag Access`
lists: One has no emptytext while the other does. Opening them in an
alternating fashion adds the empty text from one to the other.

Fix it by constructing the initial items list in initComponent, and let
the widget modify that instead.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 www/manager6/form/ListField.js | 46 +++++++++++++++++-----------------
 1 file changed, 23 insertions(+), 23 deletions(-)

diff --git a/www/manager6/form/ListField.js b/www/manager6/form/ListField.js
index 49651bc5..6fcbdfd8 100644
--- a/www/manager6/form/ListField.js
+++ b/www/manager6/form/ListField.js
@@ -124,34 +124,34 @@ Ext.define('PVE.form.ListField', {
         },
     },
 
-    items: [
-        {
-            xtype: 'grid',
-            reference: 'grid',
+    initComponent: function () {
+        let me = this;
 
-            viewConfig: {
-                deferEmptyText: false,
-            },
+        me.items = [
+            {
+                xtype: 'grid',
+                reference: 'grid',
 
-            store: {
-                listeners: {
-                    update: function () {
-                        this.commitChanges();
+                viewConfig: {
+                    deferEmptyText: false,
+                },
+
+                store: {
+                    listeners: {
+                        update: function () {
+                            this.commitChanges();
+                        },
                     },
                 },
             },
-        },
-        {
-            xtype: 'button',
-            text: gettext('Add'),
-            iconCls: 'fa fa-plus-circle',
-            handler: 'addLine',
-            margin: '5 0 0 0',
-        },
-    ],
-
-    initComponent: function () {
-        let me = this;
+            {
+                xtype: 'button',
+                text: gettext('Add'),
+                iconCls: 'fa fa-plus-circle',
+                handler: 'addLine',
+                margin: '5 0 0 0',
+            },
+        ];
 
         for (const [key, value] of Object.entries(me.gridConfig ?? {})) {
             me.items[0][key] = value;
-- 
2.47.3

[PATCH manager 2/4] ui: form: list field: fix type for header text assignment

[Dominik Csapak]

it's `fieldTitle` not `fieldTtitle`. This had the effect that no header
was shown, since no column had one. With this fix, the header is now
shown.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 www/manager6/form/ListField.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/www/manager6/form/ListField.js b/www/manager6/form/ListField.js
index 6fcbdfd8..3152c626 100644
--- a/www/manager6/form/ListField.js
+++ b/www/manager6/form/ListField.js
@@ -159,7 +159,7 @@ Ext.define('PVE.form.ListField', {
 
         me.items[0].columns = [
             {
-                header: me.fieldTtitle,
+                header: me.fieldTitle,
                 dataIndex: 'item',
                 xtype: 'widgetcolumn',
                 widget: {
-- 
2.47.3

[PATCH manager 3/4] ui: dc: options: add emptytext to user tag access edit grid

[Dominik Csapak]

This was missing but sometimes accidentally included due to a bug in the
list component. Add it to be consistent with the registered tags edit
window.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 www/manager6/dc/UserTagAccessEdit.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/www/manager6/dc/UserTagAccessEdit.js b/www/manager6/dc/UserTagAccessEdit.js
index 9d0fe13b..f2c74c89 100644
--- a/www/manager6/dc/UserTagAccessEdit.js
+++ b/www/manager6/dc/UserTagAccessEdit.js
@@ -75,6 +75,7 @@ Ext.define('PVE.dc.UserTagAccessEdit', {
                     gridConfig: {
                         height: 200,
                         scrollable: true,
+                        emptyText: gettext('No Tags defined'),
                     },
                     listeners: {
                         change: 'tagChange',
-- 
2.47.3

[PATCH manager 4/4] ui: dc options: add missing field title to registered tag edit grid

[Dominik Csapak]

This header wasn't shown by accident, so it never surfaced that this was
missing. Since the header is now visible, add the proper title.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 www/manager6/dc/RegisteredTagsEdit.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/www/manager6/dc/RegisteredTagsEdit.js b/www/manager6/dc/RegisteredTagsEdit.js
index 29aefa28..6e93c223 100644
--- a/www/manager6/dc/RegisteredTagsEdit.js
+++ b/www/manager6/dc/RegisteredTagsEdit.js
@@ -54,6 +54,7 @@ Ext.define('PVE.dc.RegisteredTagsEdit', {
                 {
                     name: 'tags',
                     xtype: 'pveListField',
+                    fieldTitle: gettext('Tag'),
                     maskRe: PVE.Utils.tagCharRegex,
                     gridConfig: {
                         height: 200,
-- 
2.47.3