🐛 Add tooltip for weight column

[ibolton336]

• Adds tooltip to weight answer table column https://issues.redhat.com/browse/MTA-1351?filter=-1

[codecov]

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (c5d050a) 41.07% compared to head (91a8a97) 41.07%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1433   +/-   ##
=======================================
  Coverage   41.07%   41.07%           
=======================================
  Files         139      139           
  Lines        4404     4404           
  Branches     1010     1010           
=======================================
  Hits         1809     1809           
  Misses       2583     2583           
  Partials       12       12           

Flag	Coverage Δ
client	41.07% <ø> (ø)
server	∅ <ø> (∅)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sjd78]

I recommend the style change, but it isn't a blocker.

[sjd78]

Since the tooltip prop can be a ReactNode, I'd rather see a better styled tooltip:

tooltip: (
  <TextContent>
    <Text>This column shows the associated risk weight of an answered question.</Text>
    <List isPlain>
      <ListItem>{getIconByRisk("red")} is High risk</ListItem>
      <ListItem>{getIconByRisk("yellow")} is Medium risk</ListItem>
      <ListItem>{getIconByRisk("green")} is Low risk</ListItem>
    </List>
  </TextContent>
),

Doing some i18n and adding the messages to translation.json would be nice.

[ibolton336]

For some reason this is the result of the tooltip with ReactNode elements

[JustinXHale]

• Adds tooltip to weight answer table column https://issues.redhat.com/browse/MTA-1351?filter=-1

this looks great

[sjd78]

Given @JustinXHale 's approval, this is ok with me. I still recommend the tooltip content change, but it is not a blocker.

[ibolton336]