apply review

Author: schultek

site/lib/_sass/components/_tooltip.scss

@@ -1,57 +1,61 @@
 .tooltip-wrapper {
-    position: relative;
-
-    a.tooltip-target {
-        text-decoration: underline dotted;
+  position: relative;
+
+  a.tooltip-target:has(+.tooltip) {
+    text-decoration: underline dotted;
+  }
+
+  .tooltip {
+    visibility: hidden;
+
+    display: flex;
+    position: absolute;
+    z-index: var(--site-z-floating);
+    top: 100%;
+    left: 50%;
+    transform: translateX(-50%);
+
+    flex-flow: column nowrap;
+    width: 20rem;
+
+    background: var(--site-raised-bgColor);
+    border: 0.05rem solid rgba(0, 0, 0, .125);
+    border-radius: 0.75rem;
+    box-shadow: 0 0.5rem 1rem rgba(0, 0, 0, .15);
+    padding: 0.8rem;
+
+    font-size: 1rem;
+    font-weight: normal;
+    font-style: normal;
+
+    .tooltip-header {
+      font-size: 1.2rem;
+      font-weight: 500;
+      margin-bottom: 0.25rem;
     }
 
-    .tooltip {
-        visibility: hidden;
-
-        display: flex;
-        position: absolute;
-        z-index: 100;
-        top: 100%;
-        left: 50%;
-        transform: translateX(-50%);
-
-        flex-flow: column nowrap;
-        width: 20rem;
-
-        background: var(--site-raised-bgColor);
-        border: 0.05rem solid rgba(0, 0, 0, .125);
-        border-radius: 0.75rem;
-        box-shadow: 0 0.5rem 1rem rgba(0, 0, 0, .15);
-        padding: 0.8rem;
-
-        .tooltip-header {
-            font-size: 1.2rem;
-            font-weight: 500;
-            margin-bottom: 0.25rem;
-        }
-
-        .tooltip-content {
-            font-size: 0.875rem;
-            color: var(--site-secondary-textColor);
-        }
+    .tooltip-content {
+      font-size: 0.875rem;
+      color: var(--site-secondary-textColor);
     }
+  }
 
-    // On non-touch devices, show tooltip on hover or focus.
-    @media all and not (pointer: coarse) {
-        &:hover .tooltip {
-            visibility: visible;
-        }
+  // On non-touch devices, show tooltip on hover or focus.
+  @media all and not (pointer: coarse) {
+    &:hover .tooltip {
+      visibility: visible;
+    }
 
-        &:has(.tooltip-target:focus) .tooltip {
-            visibility: visible;
-        }
+    &:focus-within .tooltip {
+      visibility: visible;
     }
+  }
 
-    // On touch devices, show tooltip on click.
-    @media all and (pointer: coarse) {
-        .tooltip.visible {
-            visibility: visible;
-        }
+  // On touch devices, show tooltip on click.
+  @media all and (pointer: coarse) {
+    .tooltip.visible {
+      visibility: visible;
     }
+  }
 
 }

site/lib/src/components/common/client/api_link_tooltip.dart

@@ -46,6 +46,10 @@ class _InteractiveApiLinkState extends State<ApiLinkTooltip> {
       component.url,
     );
 
+    if (extractedHeader == null && extractedDescription == null) {
+      return;
+    }
+
     if (!mounted) return;
     setState(() {
       tooltipContent = fragment([
@@ -131,7 +135,7 @@ class _InteractiveApiLinkState extends State<ApiLinkTooltip> {
         href: component.url,
         classes: 'tooltip-target',
         events: {
-          if (isTouchscreen)
+          if (tooltipContent != null && isTouchscreen)
             'click': (event) {
               if (!isVisible) {
                 setState(() => isVisible = true);

site/lib/src/style_hash.dart

@@ -2,4 +2,4 @@
 // dart format off
 
 /// The generated hash of the `main.css` file.
-const generatedStylesHash = 'XbLzy+N6PeGl';
+const generatedStylesHash = 'ukol6oECbi3R';