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

Reply via email to