all lists on lists.proxmox.com
 help / color / mirror / Atom feed
From: Dominik Csapak <d.csapak@proxmox.com>
To: pdm-devel@lists.proxmox.com
Subject: [pdm-devel] [PATCH datacenter-manager/yew-comp 00/10] status row/node status cleanup + refactor
Date: Tue, 23 Sep 2025 11:51:03 +0200	[thread overview]
Message-ID: <20250923095124.1679038-1-d.csapak@proxmox.com> (raw)

While implemnting some basic PBS panels, i stumbled over our reimplemntation
of the already existing `MeterLabel` and `StatusRow` from proxmox-yew-comp.
So this series should improve on that by reusing them.

Also refactored the panel for the NodeStatus in a way that we can reuse
it for PBS as well and at the same time make it more conform to what we already
have in the 'native' PVE/PBS gui.

Note that the pdm patches depend on the yew-comp patches, so a bump is
necessary

also i based this series on top of my other two patches[0]
(but they should not interfere with each other)

0: https://lore.proxmox.com/pdm-devel/20250923094311.1612890-1-d.csapak@proxmox.com/T/#t

proxmox-yew-comp:

Dominik Csapak (8):
  status: impl conversion to classes
  status row: add option to add the icon on the right side
  meter label: make value optional
  meter label: use `StatusRow` for text row
  meter label: align the status row with baseline
  meter label: add `animated` property
  meter label: add option to align the icon on the right
  add `node_info` helper to render a consistent view of the NodeStatus

 Cargo.toml         |   3 +
 src/lib.rs         |   3 +
 src/meter_label.rs |  53 ++++++-----
 src/node_info.rs   | 222 +++++++++++++++++++++++++++++++++++++++++++++
 src/status.rs      |   8 ++
 src/status_row.rs  |  36 ++++++--
 6 files changed, 295 insertions(+), 30 deletions(-)
 create mode 100644 src/node_info.rs


proxmox-datacenter-manager:

Dominik Csapak (2):
  ui: pve: node: use `node_info` helper from yew-comp
  ui: rework status row/meter helpers

 ui/src/pve/lxc.rs           |  48 ++++++++---------
 ui/src/pve/node/overview.rs |  76 +++-----------------------
 ui/src/pve/qemu.rs          |  48 ++++++++---------
 ui/src/pve/remote.rs        | 103 +++++++++++++++++-------------------
 ui/src/pve/storage.rs       |  55 +++++++++----------
 ui/src/renderer.rs          |  63 +++++++---------------
 6 files changed, 145 insertions(+), 248 deletions(-)


Summary over all repositories:
  12 files changed, 440 insertions(+), 278 deletions(-)

-- 
Generated by git-murpp 0.8.1


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


             reply	other threads:[~2025-09-23  9:51 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-23  9:51 Dominik Csapak [this message]
2025-09-23  9:51 ` [pdm-devel] [PATCH yew-comp 1/8] status: impl conversion to classes Dominik Csapak
2025-09-23  9:51 ` [pdm-devel] [PATCH yew-comp 2/8] status row: add option to add the icon on the right side Dominik Csapak
2025-09-23  9:51 ` [pdm-devel] [PATCH yew-comp 3/8] meter label: make value optional Dominik Csapak
2025-09-23  9:51 ` [pdm-devel] [PATCH yew-comp 4/8] meter label: use `StatusRow` for text row Dominik Csapak
2025-09-23  9:51 ` [pdm-devel] [PATCH yew-comp 5/8] meter label: align the status row with baseline Dominik Csapak
2025-09-23  9:51 ` [pdm-devel] [PATCH yew-comp 6/8] meter label: add `animated` property Dominik Csapak
2025-09-23  9:51 ` [pdm-devel] [PATCH yew-comp 7/8] meter label: add option to align the icon on the right Dominik Csapak
2025-09-23  9:51 ` [pdm-devel] [PATCH yew-comp 8/8] add `node_info` helper to render a consistent view of the NodeStatus Dominik Csapak
2025-09-23  9:51 ` [pdm-devel] [PATCH datacenter-manager 1/2] ui: pve: node: use `node_info` helper from yew-comp Dominik Csapak
2025-09-23  9:51 ` [pdm-devel] [PATCH datacenter-manager 2/2] ui: rework status row/meter helpers Dominik Csapak

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250923095124.1679038-1-d.csapak@proxmox.com \
    --to=d.csapak@proxmox.com \
    --cc=pdm-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal