Conversation
Co-authored-by: Curtis Conard <cconard96@gmail.com>
Co-authored-by: Curtis Conard <cconard96@gmail.com>
Co-authored-by: Curtis Conard <cconard96@gmail.com>
Co-authored-by: Curtis Conard <cconard96@gmail.com>
Co-authored-by: Curtis Conard <cconard96@gmail.com>
Co-authored-by: Curtis Conard <cconard96@gmail.com>
Co-authored-by: Curtis Conard <cconard96@gmail.com>
There was a problem hiding this comment.
1) The tooltip is partially hidden by other items:
I would expect it to be displayed over them, what do you think ?
2) The tooltip content is built using tables.
GLPI used to overuse tables, we have mostly corrected that now and are thus avoiding tables when displaying non-tabular data.
3) IMO the tooltip is displayed too fast.
I don't want to see it every time I move my mouse over an item, or when I am doing others actions like drawing edges or moving nodes.
There should be something like a 0.3 to 0.5s delay before the content is displayed (this seems to be the standard for this kind of UX delay).
Maybe also something to prevent the tooltip from triggered when moving nodes ?
4) As stated before the content should come from an AJAX request IMO.
This is better for performances, the new code will load a lot of individual objects one by one which may be bad for big graphes.
It also allow to generate the content using twig templates instead of raw DOM computation from javascript.
Add function for tooptip Add menu for see tooptip in getContextMenuItems Add function for cleanPopper on grab & load another tooltip
|
I do commit Change z-index Can you check ? |
|
With the latest changes I don't have any tooltips, there seems to be an issue somewhere. |
When you click on informations menu, you must move your mouse on the icon object to see tooltip |
|
Ok but what is the best approach so ? |


Impact add tooltip for nodes :