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

Aaron Lauterer a.lauterer at proxmox.com
Wed Mar 8 13:09:17 CET 2023


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

changes since

v2:
* don't use arrow functions -> rework context in many places. mainly use
  'this' now
* destroy tooltip when no detail text is present. This can easily be
  tested by setting 'ceph osd set noout' for example
* guard show/hide on info button with '?'

v1:
* change creation of info window to use autoShow
* rework tooltip to:
  * only destroy it when leaving the whole grid, use show/hide for all
    other situations
  * factor out text handling of newlines and max lines
  * factor out updating and destroying into their own functions
  * add more listeners
  * hide it when hovering the info button
  * auto update it whenever the store is updated

Aaron Lauterer (2):
  ui: ceph: make the warning detail button stand out more
  ui: ceph status: add tooltip with details to warnings

 www/manager6/ceph/Status.js | 129 ++++++++++++++++++++++++++++--------
 1 file changed, 103 insertions(+), 26 deletions(-)

-- 
2.30.2






More information about the pve-devel mailing list