Tooltips are now conditional on the hierarchy

[kinow]

Reasons for creating this PR

Link to relevant issue(s), if any

• Addresses Replace qtip JS library by a pure CSS tooltip #1324 (comment)

Description of the changes in this PR

Hierarchy is not always available. When it is not, we want to display a tooltip with a warning/note about it. However, the tooltips were always enabled, causing an empty tooltip sometimes.

Known problems or uncertainties in this PR

Hard to test. I looked at the previous modified files in the linked PR, then repeated the same fix. Tested on Firefox with the :hover modified on the hierarchy tab. In one case the tooltip is displayed as hierarchy is disabled, and in the other nothing is shown.

Hopefully that fixes the issue. Sorry!

Checklist

• phpUnit tests pass locally with my changes
• I have added tests that show that the new code works, or tests are not relevant for this PR (e.g. only HTML/CSS changes)
• The PR doesn't reduce accessibility of the front-end code (e.g. tab focus, scaling to different resolutions, use of .sr-only class, color contrast)
• The PR doesn't introduce unintended code changes (e.g. empty lines or useless reindentation)

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
2.0% Duplication

[codecov]

Codecov Report

Base: 71.17% // Head: 71.17% // No change to project coverage 👍

Coverage data is based on head (6dc902d) compared to base (77eb4a2).
Patch has no changes to coverable lines.

Additional details and impacted files

@@            Coverage Diff            @@
##             master    #1371   +/-   ##
=========================================
  Coverage     71.17%   71.17%           
  Complexity     1650     1650           
=========================================
  Files            32       32           
  Lines          3788     3788           
=========================================
  Hits           2696     2696           
  Misses         1092     1092           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[osma]

Wow, thanks a lot for the quick fix @kinow ! I will test this and if it works OK, I will merge it.

[kinow]

Wow, thanks a lot for the quick fix @kinow ! I will test this and if it works OK, I will merge it.

Not a problem @osma. Good to fix this while it's still fresh on my (normally short-term) memory. Let me know if that works 👍 Thanks!

[osma]

Works perfectly! Thanks a lot @kinow !