all lists on lists.proxmox.com
 help / color / mirror / Atom feed
From: "Shannon Sterz" <s.sterz@proxmox.com>
To: "Shannon Sterz" <s.sterz@proxmox.com>, <pve-devel@lists.proxmox.com>
Subject: Re: [pve-devel] [PATCH manager 0/9] remove all blurry icons
Date: Tue, 26 Aug 2025 17:04:10 +0200	[thread overview]
Message-ID: <DCCG9H0DD0AG.1CQAHQS6JX666@proxmox.com> (raw)
In-Reply-To: <20250826145836.302748-1-s.sterz@proxmox.com>

ah sorry should have mentioned this in the cover letter but will add
this here now:

originally this was based on my patch for just the mountpoint and guest
icons [1]. however, this has since change so extensively that calling
this series a v2 feels wrong. hope that is ok.

[1]: https://lore.proxmox.com/all/20250804105024.117020-1-s.sterz@proxmox.com/

On Tue Aug 26, 2025 at 4:58 PM CEST, 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
>
> the first patch in this series specifically addresses the icons
> mentioned in the associated bug report [1]. the next seven patches
> replace png icons and a gif spinner with svg version. the last patch
> removes entirely unused icons.
>
> i checked if the icons that are being removed were used outside of this
> repository, but could not find any other users. so this should hopefully
> not break anything.
>
> [1]: https://bugzilla.proxmox.com/show_bug.cgi?id=6599
>
> Shannon Sterz (9):
>   fix #6599: ui: use font-awesome hdd icon instead of png
>   ui: remove all occurences of icon-display.png
>   ui: replace the gif spinner from extjs with an svg
>   ui: use svg version of the noVnc icon
>   ui: use svg version of the virt viewer icon
>   ui: use svg version of xterm.js logo instead of a png
>   ui: use the svg version for the cd icon in a storage's treelist
>   ui: replace the ceph logo png with an svg version
>   ui: remove unused png icons and their asset files
>
>  www/css/ext6-pve.css                   |  91 +----
>  www/images/Makefile                    |  81 +---
>  www/images/blank.gif                   | Bin 49 -> 0 bytes
>  www/images/cdrom.png                   | Bin 960 -> 0 bytes
>  www/images/checked.png                 | Bin 478 -> 0 bytes
>  www/images/computer-on.png             | Bin 585 -> 0 bytes
>  www/images/computer-template.png       | Bin 994 -> 0 bytes
>  www/images/computer.png                | Bin 723 -> 0 bytes
>  www/images/connect_established.png     | Bin 704 -> 0 bytes
>  www/images/display.png                 | Bin 741 -> 0 bytes
>  www/images/drive-harddisk.png          | Bin 847 -> 0 bytes
>  www/images/forward.png                 | Bin 641 -> 0 bytes
>  www/images/gtk-stop.png                | Bin 664 -> 0 bytes
>  www/images/icon-cd.png                 | Bin 309 -> 0 bytes
>  www/images/icon-display.png            | Bin 221 -> 0 bytes
>  www/images/icon-harddisk.png           | Bin 293 -> 0 bytes
>  www/images/icon-keyboard.png           | Bin 309 -> 0 bytes
>  www/images/icon-network.png            | Bin 245 -> 0 bytes
>  www/images/icon-swap.png               | Bin 324 -> 0 bytes
>  www/images/icon-usb.png                | Bin 367 -> 0 bytes
>  www/images/keyboard.png                | Bin 590 -> 0 bytes
>  www/images/logo-ceph.png               | Bin 488 -> 0 bytes
>  www/images/logo-ceph.svg               |  71 ++++
>  www/images/lxc-containers-logo-off.xcf | Bin 42654 -> 0 bytes
>  www/images/lxc-off.png                 | Bin 891 -> 0 bytes
>  www/images/lxc-on.png                  | Bin 872 -> 0 bytes
>  www/images/memory.png                  | Bin 349 -> 0 bytes
>  www/images/network-server-off.png      | Bin 833 -> 0 bytes
>  www/images/network-server-on.png       | Bin 845 -> 0 bytes
>  www/images/network-server.png          | Bin 832 -> 0 bytes
>  www/images/network.png                 | Bin 703 -> 0 bytes
>  www/images/novnc.png                   | Bin 453 -> 0 bytes
>  www/images/novnc.svg                   | 163 +++++++++
>  www/images/openvz-off.png              | Bin 516 -> 0 bytes
>  www/images/openvz-on.png               | Bin 527 -> 0 bytes
>  www/images/processor.png               | Bin 635 -> 0 bytes
>  www/images/snapshot.png                | Bin 570 -> 0 bytes
>  www/images/spinner.svg                 |  24 ++
>  www/images/start.png                   | Bin 549 -> 0 bytes
>  www/images/stop.png                    | Bin 370 -> 0 bytes
>  www/images/swap.png                    | Bin 926 -> 0 bytes
>  www/images/tigervnc.png                | Bin 793 -> 0 bytes
>  www/images/unchecked.png               | Bin 322 -> 0 bytes
>  www/images/virt-viewer.png             | Bin 662 -> 0 bytes
>  www/images/virt-viewer.svg             | 488 +++++++++++++++++++++++++
>  www/images/xtermjs.png                 | Bin 729 -> 0 bytes
>  www/images/xtermjs.svg                 |  11 +
>  www/manager6/dc/Tasks.js               |   6 +-
>  www/manager6/grid/PoolMembers.js       |   4 +-
>  www/manager6/lxc/Resources.js          |   2 +-
>  www/manager6/tree/ResourceTree.js      |   1 -
>  51 files changed, 793 insertions(+), 149 deletions(-)
>  delete mode 100644 www/images/blank.gif
>  delete mode 100644 www/images/cdrom.png
>  delete mode 100644 www/images/checked.png
>  delete mode 100755 www/images/computer-on.png
>  delete mode 100644 www/images/computer-template.png
>  delete mode 100644 www/images/computer.png
>  delete mode 100644 www/images/connect_established.png
>  delete mode 100644 www/images/display.png
>  delete mode 100644 www/images/drive-harddisk.png
>  delete mode 100644 www/images/forward.png
>  delete mode 100644 www/images/gtk-stop.png
>  delete mode 100644 www/images/icon-cd.png
>  delete mode 100644 www/images/icon-display.png
>  delete mode 100644 www/images/icon-harddisk.png
>  delete mode 100644 www/images/icon-keyboard.png
>  delete mode 100644 www/images/icon-network.png
>  delete mode 100644 www/images/icon-swap.png
>  delete mode 100644 www/images/icon-usb.png
>  delete mode 100644 www/images/keyboard.png
>  delete mode 100644 www/images/logo-ceph.png
>  create mode 100644 www/images/logo-ceph.svg
>  delete mode 100755 www/images/lxc-containers-logo-off.xcf
>  delete mode 100644 www/images/lxc-off.png
>  delete mode 100644 www/images/lxc-on.png
>  delete mode 100644 www/images/memory.png
>  delete mode 100644 www/images/network-server-off.png
>  delete mode 100644 www/images/network-server-on.png
>  delete mode 100644 www/images/network-server.png
>  delete mode 100644 www/images/network.png
>  delete mode 100644 www/images/novnc.png
>  create mode 100644 www/images/novnc.svg
>  delete mode 100755 www/images/openvz-off.png
>  delete mode 100755 www/images/openvz-on.png
>  delete mode 100644 www/images/processor.png
>  delete mode 100644 www/images/snapshot.png
>  create mode 100644 www/images/spinner.svg
>  delete mode 100644 www/images/start.png
>  delete mode 100644 www/images/stop.png
>  delete mode 100644 www/images/swap.png
>  delete mode 100644 www/images/tigervnc.png
>  delete mode 100644 www/images/unchecked.png
>  delete mode 100644 www/images/virt-viewer.png
>  create mode 100644 www/images/virt-viewer.svg
>  delete mode 100644 www/images/xtermjs.png
>  create mode 100644 www/images/xtermjs.svg
>
> --
> 2.47.2



_______________________________________________
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-26 15:04 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 ` Shannon Sterz [this message]
2025-08-28 21:45 ` [pve-devel] [PATCH manager 0/9] remove all blurry icons Thomas Lamprecht
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=DCCG9H0DD0AG.1CQAHQS6JX666@proxmox.com \
    --to=s.sterz@proxmox.com \
    --cc=pve-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