public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Christoph Heiss <c.heiss@proxmox.com>
To: Proxmox VE development discussion <pve-devel@lists.proxmox.com>
Subject: Re: [pve-devel] [PATCH font-logos/manager v2 0/5] fix #2435: lxc: show distro and privileged status in summary
Date: Wed, 11 Oct 2023 16:41:28 +0200	[thread overview]
Message-ID: <3mfnfo42sx7ebjgiz235krmlmhq54mmnskpsjmswytq7fmzuhs@l22hlwzv7udo> (raw)
In-Reply-To: <20230705111257.759836-1-c.heiss@proxmox.com>


Ping - still applies cleanly on current master of both repositories.

On Wed, Jul 05, 2023 at 01:12:44PM +0200, Christoph Heiss wrote:
>
> This implements #2435 [0]. Show the unprivileged status in the summary
> panel, the distro logo and name in the title of the summary panel.
>
> Patch 1 & 2 fix two small typos in the `fonts-font-logos` package.
> Patch 3 then prepares the pveproxy to serve the required CSS and font
> files for the icon font we use [1] (packaged here [2]), and finally
> patch 4 & 5 finally wire everything up into the UI.
>
> N.B.: The `fonts-font-logos` package is not yet available through the
> repos yet (as far as I could see), so it must be built & installed
> locally for testing.
>
> [0] https://bugzilla.proxmox.com/show_bug.cgi?id=2435
> [1] https://github.com/Lukas-W/font-logos
> [2] https://git.proxmox.com/?p=fonts-font-logos.git;a=summary
>
> fonts-font-logos:
>
> Christoph Heiss (2):
>   d/install: fix typo in css install path
>   css: fix missing `@` for font-face rule
>
>  debian/install     | 2 +-
>  src/font-logos.css | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> pve-manager:
>
> Christoph Heiss (3):
>   pveproxy, ui, d/control: add font-logos
>   ui: GuestStatusView: show privileged status as new row
>   ui: GuestStatusView: show distro logo and name in summary header
>
>  PVE/Service/pveproxy.pm               |  2 ++
>  debian/control                        |  1 +
>  www/index.html.tpl                    |  1 +
>  www/manager6/panel/GuestStatusView.js | 51 ++++++++++++++++++++++++++-
>  4 files changed, 54 insertions(+), 1 deletion(-)
>
> --
> 2.39.2
>
>
>
> _______________________________________________
> pve-devel mailing list
> pve-devel@lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
>
>




  parent reply	other threads:[~2023-10-11 14:42 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-07-05 11:12 Christoph Heiss
2023-07-05 11:12 ` [pve-devel] [PATCH font-logos v2 1/5] d/install: fix typo in css install path Christoph Heiss
2023-07-05 11:12 ` [pve-devel] [PATCH font-logos v2 2/5] css: fix missing `@` for font-face rule Christoph Heiss
2023-07-05 11:12 ` [pve-devel] [PATCH manager v2 3/5] pveproxy, ui, d/control: add font-logos Christoph Heiss
2023-07-05 11:12 ` [pve-devel] [PATCH manager v2 4/5] ui: GuestStatusView: show privileged status as new row Christoph Heiss
2023-10-20 13:27   ` Dominik Csapak
2023-07-05 11:12 ` [pve-devel] [PATCH manager v2 5/5] ui: GuestStatusView: show distro logo and name in summary header Christoph Heiss
2023-10-20 13:31   ` Dominik Csapak
2023-10-11 14:41 ` Christoph Heiss [this message]
2023-10-20 13:33 ` [pve-devel] [PATCH font-logos/manager v2 0/5] fix #2435: lxc: show distro and privileged status in summary Dominik Csapak
2023-10-23 15:32   ` [pve-devel] applied-series: " Thomas Lamprecht

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=3mfnfo42sx7ebjgiz235krmlmhq54mmnskpsjmswytq7fmzuhs@l22hlwzv7udo \
    --to=c.heiss@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal