all lists on lists.proxmox.com
 help / color / mirror / Atom feed
From: Thomas Lamprecht <t.lamprecht@proxmox.com>
To: pve-devel@lists.proxmox.com, Shannon Sterz <s.sterz@proxmox.com>
Subject: Re: [pve-devel] [PATCH manager 0/9] remove all blurry icons
Date: Thu, 28 Aug 2025 23:45:21 +0200	[thread overview]
Message-ID: <175641752051.4106326.10129002106841593352.b4-ty@proxmox.com> (raw)
In-Reply-To: <20250826145836.302748-1-s.sterz@proxmox.com>

On Tue, 26 Aug 2025 16:58:27 +0200, Shannon Sterz wrote:
> this series aims to remove and replace all blurry icons. all png-based
> icons have either been:
> 
> a) removed because they were not used throught the code-base in the
>    first place
> b) adapted into svg-based versions, which replace the png versions or
> c) removed and the code was adapted to use the already existing
>    font-awesome icons
> 
> [...]

Applied, thanks!

btw. nice solution for the spinner! Can't we override the CSS classes
that have a `background-image:url(images/loadmask/loading.gif);` rule in
side our CSS file from proxmox-widget-toolkit?

FWIW, I saw other loading.gif's too, e.g. images/tree/loading.gif), that
could be checked.

[1/9] fix #6599: ui: use font-awesome hdd icon instead of png
      commit: 470dd9057ffbcde6e09fb9d00d13a882ffa915c1
[2/9] ui: remove all occurences of icon-display.png
      commit: a11f7c02e554f74488d2d0d6ceaea177e48dc552
[3/9] ui: replace the gif spinner from extjs with an svg
      commit: 61814591bc59f83562e977483629d334c67e38e0
[4/9] ui: use svg version of the noVnc icon
      commit: af50fa5836b67c1173cd3f3c55ecc8077c567802
[5/9] ui: use svg version of the virt viewer icon
      commit: 53cf0269a2b908f79ee48d66bfca71afb8e38e34
[6/9] ui: use svg version of xterm.js logo instead of a png
      commit: 1b69822949e956e66e9f1824387da6e834cc1bd5
[7/9] ui: use the svg version for the cd icon in a storage's treelist
      commit: fd40039e7bd6da90560736098597912a158d24d1
[8/9] ui: replace the ceph logo png with an svg version
      commit: 2348790ba969992dfd8235ad291e10c2823a9ebc
[9/9] ui: remove unused png icons and their asset files
      commit: 1b061a5d3268bc5904796527cec6e3c47413602c


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


  parent reply	other threads:[~2025-08-28 21:49 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-26 14:58 Shannon Sterz
2025-08-26 14:58 ` [pve-devel] [PATCH manager 1/9] fix #6599: ui: use font-awesome hdd icon instead of png Shannon Sterz
2025-08-26 14:58 ` [pve-devel] [PATCH manager 2/9] ui: remove all occurences of icon-display.png Shannon Sterz
2025-08-26 14:58 ` [pve-devel] [PATCH manager 3/9] ui: replace the gif spinner from extjs with an svg Shannon Sterz
2025-08-26 14:58 ` [pve-devel] [PATCH manager 4/9] ui: use svg version of the noVnc icon Shannon Sterz
2025-08-26 14:58 ` [pve-devel] [PATCH manager 5/9] ui: use svg version of the virt viewer icon Shannon Sterz
2025-08-26 14:58 ` [pve-devel] [PATCH manager 6/9] ui: use svg version of xterm.js logo instead of a png Shannon Sterz
2025-08-26 14:58 ` [pve-devel] [PATCH manager 7/9] ui: use the svg version for the cd icon in a storage's treelist Shannon Sterz
2025-08-26 14:58 ` [pve-devel] [PATCH manager 8/9] ui: replace the ceph logo png with an svg version Shannon Sterz
2025-08-26 14:58 ` [pve-devel] [PATCH manager 9/9] ui: remove unused png icons and their asset files Shannon Sterz
2025-08-26 15:04 ` [pve-devel] [PATCH manager 0/9] remove all blurry icons Shannon Sterz
2025-08-28 21:45 ` Thomas Lamprecht [this message]
2025-08-29  9:26   ` 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=175641752051.4106326.10129002106841593352.b4-ty@proxmox.com \
    --to=t.lamprecht@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.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal