Bugfixes breakdown chart

Vizzuality · Dec 11, 2024 · 650ca5c · 650ca5c
1 parent b94b13d
commit 650ca5c
Show file tree

Hide file tree

Showing 5 changed files with 31 additions and 32 deletions.
diff --git a/client/src/containers/header/index.tsx b/client/src/containers/header/index.tsx
@@ -2,6 +2,8 @@
 
 import { FC, useState } from "react";
 
+import Link from "next/link";
+
 import { Menu as MenuIcon } from "lucide-react";
 
 import { Overlay } from "@/components/ui/overlay";
@@ -12,7 +14,6 @@ import {
 } from "@/components/ui/popover";
 
 import Menu from "./menu";
-import Link from "next/link";
 
 const Header: FC = () => {
   const [isOpen, setIsOpen] = useState(false);

diff --git a/client/src/containers/header/menu/index.tsx b/client/src/containers/header/menu/index.tsx
@@ -1,9 +1,9 @@
 import { FC } from "react";
 
 import Link from "next/link";
+import { usePathname } from "next/navigation";
 
 import UserMenu from "./user";
-import { usePathname } from "next/navigation";
 
 export const classes = {
   link: "block px-4 py-2 hover:bg-muted transition-colors",

diff --git a/client/src/containers/widget/index.tsx b/client/src/containers/widget/index.tsx
@@ -149,12 +149,7 @@ export default function Widget({
           className,
         )}
       >
-        <WidgetHeader
-          indicator={indicator}
-          question={question}
-          menu={menu}
-          responseRate={responseRate}
-        />
+        <WidgetHeader indicator={indicator} question={question} menu={menu} />
         <Breakdown data={data.breakdown} />
       </Card>
     );

diff --git a/client/src/containers/widget/widget-header/index.tsx b/client/src/containers/widget/widget-header/index.tsx
@@ -33,26 +33,27 @@ const WidgetHeader: FC<WidgetHeaderProps> = ({
         <Title as="h3" className="text-base">
           {indicator}
         </Title>
-        <div className="flex items-center gap-2">
-          <TooltipProvider>
-            <Tooltip delayDuration={350}>
-              <TooltipTrigger>
-                <p
-                  data-testid="response-rate"
-                  className="inline-flex items-center gap-1 rounded-full bg-primary p-2 text-2xs font-medium text-muted-foreground"
-                >
-                  <ClipboardCheckIcon className="h-4 w-4" strokeWidth={1.5} />
-                  <span>{responseRate}%</span>
-                </p>
-              </TooltipTrigger>
-              <TooltipContent className="rounded-full border-none bg-background px-2 py-1 text-xs text-foreground">
-                <p>Response rate</p>
-              </TooltipContent>
-            </Tooltip>
-          </TooltipProvider>
-
-          {menu}
-        </div>
+        {typeof responseRate === "number" && (
+          <div className="flex items-center gap-2">
+            <TooltipProvider>
+              <Tooltip delayDuration={350}>
+                <TooltipTrigger>
+                  <p
+                    data-testid="response-rate"
+                    className="inline-flex items-center gap-1 rounded-full bg-primary p-2 text-2xs font-medium text-muted-foreground"
+                  >
+                    <ClipboardCheckIcon className="h-4 w-4" strokeWidth={1.5} />
+                    <span>{responseRate}%</span>
+                  </p>
+                </TooltipTrigger>
+                <TooltipContent className="rounded-full border-none bg-background px-2 py-1 text-xs text-foreground">
+                  <p>Response rate</p>
+                </TooltipContent>
+              </Tooltip>
+            </TooltipProvider>
+          </div>
+        )}
+        {menu}
       </div>
       {question && <p className="text-xs text-muted-foreground">{question}</p>}
     </header>

diff --git a/client/src/lib/normalize-widget-data.ts b/client/src/lib/normalize-widget-data.ts
@@ -19,10 +19,12 @@ function normalizeWidgetData(widgetData: WidgetData): WidgetData {
   }
 
   if (result.breakdown) {
-    result.breakdown = result.breakdown.map((b) => ({
-      ...b,
-      data: normalizeChartData(b.data),
-    }));
+    result.breakdown = result.breakdown
+      .map((b) => ({
+        ...b,
+        data: normalizeChartData(b.data),
+      }))
+      .filter((d) => d.label !== "N/A");
   }
 
   return result;