On 10/3/19 1:50 PM, Dominik Csapak wrote: > so that the user can edit the tags in the gui
IMO the options is not really "correct", it's not a guest option at all. Rather this fits to the "Notes" field of the Summary panel, maybe finding there a place would be better.. > > Signed-off-by: Dominik Csapak <d.csa...@proxmox.com> > --- > www/manager6/lxc/Options.js | 13 +++++++++++++ > www/manager6/qemu/Options.js | 13 +++++++++++++ > 2 files changed, 26 insertions(+) > > diff --git a/www/manager6/lxc/Options.js b/www/manager6/lxc/Options.js > index 23409938..049b35e3 100644 > --- a/www/manager6/lxc/Options.js > +++ b/www/manager6/lxc/Options.js > @@ -141,6 +141,19 @@ Ext.define('PVE.lxc.Options', { > editor: Proxmox.UserName === 'root@pam' ? > 'PVE.lxc.FeaturesEdit' : undefined > }, > + tags: { > + header: gettext('Tags'), > + defaultValue: false, > + renderer: val => val || Proxmox.Utils.noneText, > + editor: caps.vms['VM.Config.Options'] ? { > + xtype: 'proxmoxWindowEdit', > + subject: gettext('Tags'), > + items: { > + xtype: 'pveTagSelector', > + name: 'tags', > + } > + } : undefined > + }, > hookscript: { > header: gettext('Hookscript') > } > diff --git a/www/manager6/qemu/Options.js b/www/manager6/qemu/Options.js > index e1580060..e17d8576 100644 > --- a/www/manager6/qemu/Options.js > +++ b/www/manager6/qemu/Options.js > @@ -281,6 +281,19 @@ Ext.define('PVE.qemu.Options', { > } > } : undefined > }, > + tags: { > + header: gettext('Tags'), > + defaultValue: false, > + renderer: val => val || Proxmox.Utils.noneText, > + editor: caps.vms['VM.Config.Options'] ? { > + xtype: 'proxmoxWindowEdit', > + subject: gettext('Tags'), > + items: { > + xtype: 'pveTagSelector', > + name: 'tags', > + } > + } : undefined > + }, > hookscript: { > header: gettext('Hookscript') > } > _______________________________________________ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel