From: Thomas Lamprecht <t.lamprecht@proxmox.com>
To: Stoiko Ivanov <s.ivanov@proxmox.com>, pmg-devel@lists.proxmox.com
Subject: [pmg-devel] applied: [RFC pmg-gui] fix #4311: add quarantine help-page for end-users
Date: Tue, 25 Feb 2025 21:30:14 +0100 [thread overview]
Message-ID: <948c53aa-9e1c-42c5-bca8-9a7d934953c3@proxmox.com> (raw)
In-Reply-To: <20250224121935.306240-1-s.ivanov@proxmox.com>
Am 24.02.25 um 13:19 schrieb Stoiko Ivanov:
> Having a short explanation of what the quarantine interface provides,
> in the end-user interface should help users who wonder why they get
> mails from some system, and what quarantine means. Instead of adding
> this to the regular documentation and linking it in the top-bar as we
> do in the admin-view having a short description directly rendered on
> the main panel should increase the chances of it getting read.
>
> the code is inspired by the notesview (I found it when clicking
> through a PBS system of mine looking for how to address this).
>
> Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
> ---
> Sending as RFC as I'm not quite sure if having this as static entry in
> the GUI is a good fit. If we want to keep it this way, I'd replace the
> `Proxmox.Markdown.parse` call with the rendered html.
>
> One alternative that came to my mind while writing this is having the
> markdown text in the backend as a template. This would give admins an
> easy way to customize this to their needs, and provide it in their
> local language.
>
> Additionally of course feedback on the (rather terse) phrasing itself
> is of course also appreciated.
>
>
> js/Makefile | 1 +
> js/QuarantineView.js | 8 +++++++
> js/QuarantineViewAbout.js | 44 +++++++++++++++++++++++++++++++++++++++
> 3 files changed, 53 insertions(+)
> create mode 100644 js/QuarantineViewAbout.js
>
>
applied, with some style and wording adaptions in a follow-up commit, thanks!
btw. Pressing the delete button does not work here in my (firefox) browser.
The short-cuts are also not really that ideal, but that's yet another topic,
at least users can now know them..
_______________________________________________
pmg-devel mailing list
pmg-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pmg-devel
prev parent reply other threads:[~2025-02-25 20:30 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-24 12:19 [pmg-devel] " Stoiko Ivanov
2025-02-25 20:30 ` Thomas Lamprecht [this message]
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=948c53aa-9e1c-42c5-bca8-9a7d934953c3@proxmox.com \
--to=t.lamprecht@proxmox.com \
--cc=pmg-devel@lists.proxmox.com \
--cc=s.ivanov@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