Re: [pve-devel] [PATCH manager v3 09/14] ui: GuestStatusView: add memhost for VM guests

[Dominik Csapak <d.csapak@proxmox.com>]

Since this is getting rather crowded, we should probably think about
how we could redesign this panel to save a bit of space
(e.g. show the status and ha status on one line, moving the node
info somewhere else, ...)

FWICT this is not that big of a problem (yet)

Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

On 7/15/25 16:32, Aaron Lauterer wrote:
> With the new memhost field, the vertical space is getting tight. We
> therefore reduce the height of the separator boxes.
> 
> Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>
> ---
>   www/manager6/panel/GuestStatusView.js | 18 ++++++++++++++++--
>   1 file changed, 16 insertions(+), 2 deletions(-)
> 
> diff --git a/www/manager6/panel/GuestStatusView.js b/www/manager6/panel/GuestStatusView.js
> index 0134526c..3369f7b3 100644
> --- a/www/manager6/panel/GuestStatusView.js
> +++ b/www/manager6/panel/GuestStatusView.js
> @@ -94,7 +94,7 @@ Ext.define('PVE.panel.GuestStatusView', {
>           },
>           {
>               xtype: 'box',
> -            height: 15,
> +            height: 10,
>           },
>           {
>               itemId: 'cpu',
> @@ -114,6 +114,20 @@ Ext.define('PVE.panel.GuestStatusView', {
>               valueField: 'mem',
>               maxField: 'maxmem',
>           },
> +        {
> +            itemId: 'memory-host',
> +            iconCls: 'fa fa-fw pmx-itype-icon-memory pmx-icon',
> +            title: gettext('Host memory usage'),
> +            valueField: 'memhost',
> +            printBar: false,
> +            renderer: function (used, max) {
> +                return Proxmox.Utils.render_size(used);
> +            },
> +            cbind: {
> +                hidden: '{isLxc}',
> +                disabled: '{isLxc}',
> +            },
> +        },
>           {
>               itemId: 'swap',
>               iconCls: 'fa fa-refresh fa-fw',
> @@ -144,7 +158,7 @@ Ext.define('PVE.panel.GuestStatusView', {
>           },
>           {
>               xtype: 'box',
> -            height: 15,
> +            height: 10,
>           },
>           {
>               itemId: 'ips',



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel