Re: [pdm-devel] [PATCH proxmox-datacenter-manager 4/5] ui: images: add sdn icon

[Stefan Hanreich <s.hanreich@proxmox.com>]

On 9/9/25 3:16 PM, Dominik Csapak wrote:
> while this is good enough for now imho
> 
> we should really think about creating our own font with the missing
> icons, or think about changing to an iconset that includes the ones we
> need/want. The approach here fixes the color issue, but we still
> have a scaling problem, as in, the svg does not scale with the
> set font-size, like the rest of the font-awesome icons.
> 
> This is ok for the uses we have now, but might not always work out
> (e.g. the font-size in panel headers is 20px for desktop and
> 14px for crisp, so the 16px here sit somewhere in between, so it
> does not stand out that much)

Yeah, I think so as well - I stated it in the commit message for that
purpose. Didn't even think of the size not applying as well tbh - that
basically extends to every property that affects text I suppose
(line-height comes to mind as potentially problematic as well for
instance). Might want to look into that sooner rather than later...


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