[pve-devel] applied: [PATCH manager] ui: tree: show tooltips for all entries that have sensible information

Thomas Lamprecht t.lamprecht at proxmox.com
Thu Nov 14 19:09:20 CET 2024


Am 14.11.24 um 09:04 schrieb Dominik Csapak:
> instead of filtering pools and groups out manually, simply accumulate
> the info we want to to have (status/hastate/etc.) and if any of those
> exist, show the tooltip.
> 
> This results in the nodes also having a tooltip, including their hastate
> (such as online/maintenance), and automatically would show such things
> in the future if we add those fields to other entries.
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>  www/manager6/tree/ResourceTree.js | 11 +++++++----
>  1 file changed, 7 insertions(+), 4 deletions(-)
> 
>

applied, thanks!

One minor thing I noticed, and where I'm rather unsure if we should do
anything about it, is some potential odd interaction between the tooltip
and the context menu, e.g. if I open the context menu of a node and then
move the cursor up to the node icon the tooltip pops up and is displayed
over the context menu and needs relatively long to hide.
But FWICT this was always the case for guests already anyway, so probably
it's just me overthinking this.




More information about the pve-devel mailing list