[pve-devel] [PATCH v3 pve-manager 59/66] ui: perm path: load notification target/filter acl entries

Thomas Lamprecht t.lamprecht at proxmox.com
Thu Jul 20 17:02:25 CEST 2023

On 20/07/2023 11:26, Maximiliano Sandoval wrote:
>> On 20.07.2023 10:29 CEST Fiona Ebner <f.ebner at proxmox.com> wrote:
>> Am 20.07.23 um 10:22 schrieb Lukas Wagner:
>>> Aaaah true, I didn't realize that. These non-obvious editable combo boxes
>>> have bugged me for a while now. Maybe I'll look into it to see if there
>>> is a way to make
>>> these a bit more obvious to use. IIRC, our schedule pickers have
>>> '(editable)' in the empty text,

Yeah I did that and am not really happy with it, and we had at least some
users that were still confused, we naturally have no numbers of people where
this empty-text alone was enough to understand it clearly, so hard to say what
the net benefit of it was.

>>> but that feels more like a workaround rather than an actual solution to
>>> that issue.
>> Maybe some kind of pencil icon on the side within the field? With an
>> "Edit" tooltip, selecting the text when clicking it? Just an idea that
>> came to mind, not sure if it's any good.

For editable dropdowns I'd might be better option to have a extra entry in
the selector, like "custom" or "manual",maybe even with the pencil icon to
distinguish it from the actual choices. If selected it can simply set the
text to '' and focus the field (i.e., just give it a value of '').

ps. can we *please* avoid top-posting, rather use interleaved posting (or
at least bottom) also trim the context of replies to what matters (applies
not only to you in this thread)

More information about the pve-devel mailing list