[pbs-devel] [pve-devel] [PATCH proxmox-widget-toolkit 2/2] disk list: disable show smart values button if status unknown

Thomas Lamprecht t.lamprecht at proxmox.com
Tue Feb 25 17:41:34 CET 2025


Am 25.02.25 um 17:22 schrieb Christian Ebner:
> Acked, will incorporate the suggestions for both patches in a v2, thanks 
> for feedback!

btw before I forget it: This is some nice UX fine-polishing, but IMO the smart
status panel is not really ideal as main disk panel that opens on double click
and so on.

I'd rather transform this to open a generic disk, or partition, information
window showing things like model, serial, (part)uuid, in a main tab and make
that easily copyable. The smart data could be moved to a second tab in such
a window and only loaded when that opened. We could then mask its grid there
if the smart status is unknown or hide it completely if it cannot be queried,
like for partitions.
Some bigger changes but IMO quite a bit better UX than the status quo, which
feels rather confusing.




More information about the pbs-devel mailing list