[pve-devel] [PATCH manager/widget-toolkit 0/2] ui: replace non-clickable checkboxes with Yes/No text

[Thomas Lamprecht]

Am 23/01/2023 um 11:46 schrieb Lukas Wagner:
> On 1/20/23 15:09, Thomas Lamprecht wrote:
>>> While looking sleek, the problem with this is that from a user's
>>> perspective, a checkbox generally implies that it is operable by
>>> clicking on it (which we allow in other places, to make the matter even
>>> more confusing).
>>
>> If it's editable it gets a pointer cursor, else not.
>> I see were you're comming from, but do we also have any complaints on official
>> channels w.r.t. this?
> 
> No complaints, it's just something that I've run into myself repeatedly, e.g. when
> enabling the no-subscription repo via the GUI. I think it's one of those UX
> problems that might be annoying to some users (including myself), however it's not
> severe enough that people care to report this in the bug tracker.
> 

Ok, tbh. I have some faint memory that I saw some comment about this in the
distant past; IIRC it was mostly due to the the "writeable" firewall and the
"read-only" other usages using both the exact same display.

> 
> I have played around a bit with FA icons, and I think I have found something that is visually
> appealing, fixed-width and where it is IMO clear that it is not an actionable UI item.
> For now, I think the nicest option is `fa-check` for enabled rows and `fa-minus` for disabled ones.
> I've created an A:B comparison [1] between the old checkboxes and the new icons.
> Please let me know what you think.
> 

looks better than the status quo, especially UX-wise, and would be an option for
icon only. So, if nobody else has hard feelings (but ideally somewhat rationally
argued) for going with text over icon I'd go for your combination check-mark/minus
icon combination.

> 
> [1] https://imgur.com/a/tsXegNF
>