Re: [pve-devel] [PATCH v3 manager 0/2] ui: ceph: improve discoverability of warning details

[Aaron Lauterer <a.lauterer@proxmox.com>]

On 3/8/23 17:05, Thomas Lamprecht wrote:
> Am 08/03/2023 um 13:09 schrieb Aaron Lauterer:
>> The goal for this small series is to make it easier/more obvious to see
>> that there are potentially more details for a warning ceph is showing
>> by:
>> * having a tooltip with the details (limited in length)
>> * making the detail/info button more visible
>>
> 
> One bug I see is that if I open the detail window, the next extjs (probably store)
> refresh seemingly refocuses the some part of the row and then the tooltip pops up
> too for the element I'm already having the full detail window open.

Thanks for catching that! :-/

> 
> That said, why not avoid the tooltip but inline the info by moving the severity to
> the right, dropping the info button completely and making the grid row expandable,
> i.e., like the PMG tracking center grid does?

We thought about it, but initially I wasn't too happy about it, because if the 
detailed error message has many lines and/or long lines, it would be nicer to 
just open the window and making it as large as possible.
With the expandable row, the visible area can be rather small.

But with that unfortunate behavior with the detail window opened, I am more 
inclined to make the rows expandable.

I'll give it a try and will send a v4 :)

> 
> Would feel more integrated and less duplicate (tooltip + detail window)
> information.