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

Lukas Wagner l.wagner at proxmox.com
Mon Jan 23 12:01:41 CET 2023



On 1/23/23 11:57, Thomas Lamprecht wrote:
> 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.
> 
My bad, there actually was a report about this, Leo kindly showed me the entry in Bugzilla just now [1].

>>
>> 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.
> 

Ok great, then I'll post a v2 with the icon-version soon.

Thanks!

[1] https://bugzilla.proxmox.com/show_bug.cgi?id=4350

-- 
- Lukas





More information about the pve-devel mailing list