From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) by lore.proxmox.com (Postfix) with ESMTPS id 77B9E1FF186 for ; Fri, 29 Aug 2025 11:26:28 +0200 (CEST) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id A99B8292F6; Fri, 29 Aug 2025 11:26:28 +0200 (CEST) Mime-Version: 1.0 Date: Fri, 29 Aug 2025 11:26:25 +0200 Message-Id: To: "Thomas Lamprecht" , X-Mailer: aerc 0.20.0 References: <20250826145836.302748-1-s.sterz@proxmox.com> <175641752051.4106326.10129002106841593352.b4-ty@proxmox.com> In-Reply-To: <175641752051.4106326.10129002106841593352.b4-ty@proxmox.com> From: "Shannon Sterz" X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1756459577643 X-SPAM-LEVEL: Spam detection results: 0 AWL -0.099 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment KAM_LOTSOFHASH 0.25 Emails with lots of hash-like gibberish SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Subject: Re: [pve-devel] [PATCH manager 0/9] remove all blurry icons X-BeenThere: pve-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox VE development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: Proxmox VE development discussion Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: pve-devel-bounces@lists.proxmox.com Sender: "pve-devel" On Thu Aug 28, 2025 at 11:45 PM CEST, Thomas Lamprecht wrote: > 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? good point, haven't tried that yet, but will give it a go. > FWIW, I saw other loading.gif's too, e.g. images/tree/loading.gif), that > could be checked. yep was also gonna check pmg and pbs for icons that are still blurry. but pve was the biggest fish there i think. but yeah, it seems that extjs is happy to include basically the same spinner gif three times (images/tree/loading.gif; images/grid/loading.gif; images/loadmask/loading.gif). btw. seems that something has gone wrong applying patches for the ceph and virt viewer icon. talked to christoph a bit and our best guess is the 998 character limit according to rfc 2822 for lines in emails. both of the svgs there have one really long path tag that seems to overrun that limit. interestingly, every part of the toolchain (format-patch, send-email, the mailing list and lore) dealt with that just fine, though. i'll send a patch that reformats them and adds them back. ps: christoph pointed me at the `--validate` option of `git send-email` that should check just that, so i guess i'll add that to my gitconfig > [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