From: Fiona Ebner <f.ebner@proxmox.com>
To: Thomas Lamprecht <t.lamprecht@proxmox.com>,
Proxmox VE development discussion <pve-devel@lists.proxmox.com>
Subject: Re: [pve-devel] [PATCH manager 1/2] ui: qemu: CD edit: add "Installation media" label when in wizard
Date: Thu, 22 Sep 2022 10:18:40 +0200 [thread overview]
Message-ID: <6133ed51-6660-9d52-6df8-38caa287c4d9@proxmox.com> (raw)
In-Reply-To: <765748c9-d21f-68c7-e6c4-8406cf0bbccd@proxmox.com>
Am 19.09.22 um 13:00 schrieb Thomas Lamprecht:
> Bit torn on this one, otoh it's small and might help some, otoh I think that
> it shouldn't be required - there's OS in the tab title already and a disk tab
> two steps ahead, that paired with the fact that there's clearly written "CD/DVD
> drive" so it should be pretty easy to understand, and if not that extra hint
> isn't probably adding that much of extra info over the existing cues, also note,
> and here I'm probably over analysing/nit-picking, but it doesn't necesarrily
> needs to be a installation media, e.g. for a live distro.
>
Yes, if somebody actually reads the text, it should be clear what the
storage is for ;)
> The heading fits somewhat with the one from the OS type selector, so from a purely
> layout POV it would be fine to have. Maybe just some other wording, like a plain/
> technical "Virtual CD/DVD Drive".
>
What about "Installation/Live media"? Should be accurate enough for the
vast majority of scenarios and I think, it'd provide a good hint to new
users. It also would avoid repeating "CD/DVD", but I'm fine with your
suggestion too :)
prev parent reply other threads:[~2022-09-22 8:19 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-19 9:16 Fiona Ebner
2022-09-19 9:16 ` [pve-devel] [PATCH/RFC manager 2/2] ui: qemu: CD edit: make label for storage more descriptive Fiona Ebner
2022-09-19 10:01 ` Thomas Lamprecht
2022-09-19 11:00 ` [pve-devel] [PATCH manager 1/2] ui: qemu: CD edit: add "Installation media" label when in wizard Thomas Lamprecht
2022-09-22 8:18 ` Fiona Ebner [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=6133ed51-6660-9d52-6df8-38caa287c4d9@proxmox.com \
--to=f.ebner@proxmox.com \
--cc=pve-devel@lists.proxmox.com \
--cc=t.lamprecht@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.