From: Dominik Csapak <d.csapak@proxmox.com>
To: Shannon Sterz <s.sterz@proxmox.com>, pve-devel@lists.proxmox.com
Subject: Re: [PATCH manager/pmg-gui/proxmox-backup v2 0/3] blurry icons follow-up
Date: Fri, 17 Apr 2026 09:21:53 +0200 [thread overview]
Message-ID: <ec3aeee5-64a7-4d11-9558-089f461a600c@proxmox.com> (raw)
In-Reply-To: <20260224121009.155272-1-s.sterz@proxmox.com>
instead of changing the image in all products, we should move
the css class (which is identical) into proxmox-widget-toolkit.
we already use this class there, so it should live there anyway.
with a bumped pwt + versioned dependency, we can then remove these
classes in the products themselves
On 2/24/26 1:10 PM, Shannon Sterz wrote:
> this series is a small follow up to the recent series that removed all
> custom blurry icons from proxmox ve [1]. parts of these series have been
> applied previously [2], these remaining patches use an svg-based spinner
> across pve-manager, pmg-gui, and proxmox-backup.
>
> [1]: https://lore.proxmox.com/all/20250826145836.302748-2-s.sterz@proxmox.com/
> [2]: https://lore.proxmox.com/all/20250829141219.290626-2-s.sterz@proxmox.com/
>
> Changelog
> ---------
>
> changes since v1:
>
> * dropped the already applied proxmox-widget-toolkit and pve-manager
> patches
> * rebased on current master for each repo
>
> pve-manager:
>
> Shannon Sterz (1):
> ui: use spinner from proxmox-widget-toolkit instead of adding it here
>
> www/css/ext6-pve.css | 2 +-
> www/images/Makefile | 1 -
> www/images/spinner.svg | 24 ------------------------
> 3 files changed, 1 insertion(+), 26 deletions(-)
> delete mode 100644 www/images/spinner.svg
>
>
> proxmox-backup:
>
> Shannon Sterz (1):
> css: replace blurry extjs spinner with widget-toolkit's svg version
>
> www/css/ext6-pbs.css | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>
> pmg-gui:
>
> Shannon Sterz (1):
> css: replace blurry extjs spinner with widget-toolkit's svg version
>
> css/ext6-pmg.css | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>
> Summary over all repositories:
> 5 files changed, 3 insertions(+), 28 deletions(-)
>
> --
> Generated by git-murpp 0.8.1
>
>
>
>
next prev parent reply other threads:[~2026-04-17 7:22 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-24 12:10 Shannon Sterz
2026-02-24 12:10 ` [PATCH manager v2 1/1] ui: use spinner from proxmox-widget-toolkit instead of adding it here Shannon Sterz
2026-02-24 12:10 ` [PATCH proxmox-backup v2 1/1] css: replace blurry extjs spinner with widget-toolkit's svg version Shannon Sterz
2026-02-24 12:10 ` [PATCH pmg-gui " Shannon Sterz
2026-03-27 9:46 ` [PATCH manager/pmg-gui/proxmox-backup v2 0/3] blurry icons follow-up Shannon Sterz
2026-04-17 7:21 ` Dominik Csapak [this message]
2026-04-17 8:26 ` Superseded: " Shannon Sterz
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=ec3aeee5-64a7-4d11-9558-089f461a600c@proxmox.com \
--to=d.csapak@proxmox.com \
--cc=pve-devel@lists.proxmox.com \
--cc=s.sterz@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.