[pve-devel] [PATCH qemu-server/container/toolkit/manager v3] fix #1934: add tags to guests

[Thomas Lamprecht]

On 11/25/19 11:33 AM, Dominik Csapak wrote:
> On 11/20/19 8:10 PM, Thomas Lamprecht wrote:
>> applied qemu-server and container, for the Webinterface part I'd like
>> to see some changes/features..
>>
>> * tag reordering (or show/save them sorted)
> 
> they already are shown in the order they are put in
> this was done to not impose any order on the user
> 
> reordering can be done by, delete and add again

not really nice UX when I have to delete 5 tags just to be able
to re-order some in the middle.. If no re-ordering support is there
I'd at least sort them alphabetically..

> a reordering (via drag drop) can surely be done, but
> will probably not be done in a day
> 
>> * editing a specific single tag
> what for though? if one misspelled a tag, the user can
> simply delete and add it again

and have another order with your approach, it's something I intutively
tried.

> 
> (a tag is not inherently something persistent so editing
> would be the same as deleting and adding a different tag)
> 
>> * tag edit window is pretty small and the UX is so lala, maybe one will
>>    accustom to it, but maybe it would be better to just have a list of
>>    tags (textfield or field-container)
> 
> we can make it bigger and as soon as we have a cluster wide list of tags, we can make use of the builtin (now disabled) drop down box
> for selecting a pre existing tag
> 
> what did you imagine with your proposal? a simple 'free form' text field?

as stated above: a list of tags (textfield or field-container), re-order
knob at the front, then edit field, then deletion X.

I'd either make it editable full in line, where an approach like your could
be OK, or if we open a window then to it like above.

> 
>> * collapse tags if there are more than X (e.g., >3 and/or configurable
>>    over browser settings), where the full list is then shown only on
>>    mouse-hover, as it gets crowded fast.
> 
> this can make sense, but also kinda defeats the purpose to show them
> in the gui in the first place (since i want to see all tags every time)

I don't, that why my "configurable over browser settings" was there ;)
I want to see them when I want, i.e., short list of tags always, other
when I hover, I don't want to fully clutter my view always.

> 
> what probably would be better is to check the available space
> (e.g. via browser width) and limit on a small monitor, but
> show all if there is much space (kind of how we decide
> to show one/two columns in the summary)
> 

better, still cluttering - there's starting to be really to much elements
visible all the time in the UX, makes things more complicate to find..