Re: [RFC PATCH datacenter-manager 2/2] ui: pve: tree: size action column better

["Lukas Wagner" <l.wagner@proxmox.com>]

On Fri Oct 2, 2026 at 3:50 PM CEST, Dominik Csapak wrote:
> instead of using a fixed pixel size. While the new expression looks ugly, that is the
> exact size needed to show all action columns in all density variants.
>
> Ideally we'd have a way to automatically size a column by it's content
> (in a buffered grid), but until we implement such a thing, this should
> be ok.
>
> Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
> ---
> meh, i don't like either the before nor the after. I have some ideas
> how we could make this work on the datatable side in a generic way, but
> this is rather involved, and i'm not sure if it's worth it to spend
> much time on it when we can do it like this.
>
>  ui/src/pve/tree.rs | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/ui/src/pve/tree.rs b/ui/src/pve/tree.rs
> index 2faa4349..6cac96fa 100644
> --- a/ui/src/pve/tree.rs
> +++ b/ui/src/pve/tree.rs
> @@ -645,7 +645,7 @@ fn columns(
>              })
>              .into(),
>          DataTableColumn::new(tr!("Actions"))
> -            .width("180px")
> +            .width("calc(4 * (20px + 2* var(--pwt-spacer-1)) + 2 * var(--pwt-spacer-2))")

Could be worth a comment IMO, until we have a nicer way to handle this
generically.

>              .render(move |entry: &PveTreeNode| {
>                  let (id, local_id, guest_info, node) = match entry {
>                      PveTreeNode::Lxc(r) => {