From 3a7aab00e88a47834cf199f1fa599ade09c8363d Mon Sep 17 00:00:00 2001
From: Jamie Barton <jamie@notrab.dev>
Date: Mon, 4 Dec 2023 12:43:12 +0000
Subject: [PATCH] feat: allow nameshield children (#175)

* feat: allow nameshield children

* bump

* more props

* bump version

* remove notrab
---
 packages/nameguard-react/package.json         |  2 +-
 .../src/components/NameBadge/NameBadge.tsx    |  8 ++++-
 .../src/components/NameShield/index.tsx       | 32 +++++++++----------
 .../src/components/Search/SearchModal.tsx     |  2 +-
 .../src/components/Shield/Shield.tsx          |  3 +-
 5 files changed, 26 insertions(+), 21 deletions(-)

diff --git a/packages/nameguard-react/package.json b/packages/nameguard-react/package.json
index 1fabdda77..6c1ceeb57 100644
--- a/packages/nameguard-react/package.json
+++ b/packages/nameguard-react/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@namehash/nameguard-react",
-  "version": "0.0.11",
+  "version": "0.0.13",
   "scripts": {
     "build": "tsup",
     "dev": "tsup --watch --clean=false",
diff --git a/packages/nameguard-react/src/components/NameBadge/NameBadge.tsx b/packages/nameguard-react/src/components/NameBadge/NameBadge.tsx
index 60535e992..edc36033c 100644
--- a/packages/nameguard-react/src/components/NameBadge/NameBadge.tsx
+++ b/packages/nameguard-react/src/components/NameBadge/NameBadge.tsx
@@ -31,7 +31,13 @@ export const NameBadge = ({ placeholder, onClick, data }: NameBadgeProps) => {
       <span className="text-black text-sm leading-5">
         {data.beautiful_name || data.name}
       </span>
-      <NameShield data={data} />
+      <NameShield data={data}>
+        <div className="text-sm text-white">
+          <button className="appearance-none underline" onClick={onClick}>
+            Inspect name for details
+          </button>
+        </div>
+      </NameShield>
     </button>
   );
 };
diff --git a/packages/nameguard-react/src/components/NameShield/index.tsx b/packages/nameguard-react/src/components/NameShield/index.tsx
index c76af414e..7cd2ba796 100644
--- a/packages/nameguard-react/src/components/NameShield/index.tsx
+++ b/packages/nameguard-react/src/components/NameShield/index.tsx
@@ -1,4 +1,4 @@
-import React from "react";
+import React, { type ReactNode } from "react";
 import type { ConsolidatedNameGuardReport, Rating } from "@namehash/nameguard";
 import cc from "classcat";
 
@@ -24,15 +24,21 @@ function textColor(rating?: Rating) {
 
 type NameShieldProps = {
   data?: ConsolidatedNameGuardReport;
+  children?: ReactNode;
+  disableHover?: boolean;
 };
 
-export function NameShield({ data }: NameShieldProps) {
+export function NameShield({ data, children, disableHover }: NameShieldProps) {
   if (!data) return null;
 
-  const { title, name, rating, risk_count, highest_risk } = data;
+  const { title, subtitle, rating, risk_count, highest_risk } = data;
 
   const textClass = cc(["font-semibold", textColor(rating)]);
 
+  if (disableHover) {
+    return <Shield status={rating} size="tiny" className="cursor-pointer" />;
+  }
+
   return (
     <Tooltip
       trigger={
@@ -48,22 +54,14 @@ export function NameShield({ data }: NameShieldProps) {
           <div className="flex items-center justify-between">
             <span className={textClass}>{title}</span>
             <span className="text-sm font-normal text-gray-400">
-              {risk_count} risk
-              {(risk_count || 0) > 1 && "s"} detected
+              {risk_count} risk{risk_count !== 1 && "s"} detected
             </span>
           </div>
-          <div className="space-y-0.5">
-            <div className="text-sm text-white">{highest_risk?.message}</div>
-            {/* <div className="text-sm text-white">
-              <a
-                href={`https://nameguard.io/inspect/${name}`}
-                className="underline"
-                rel="noopener noreferrer"
-                target="_blank"
-              >
-                Inspect name for details
-              </a>
-            </div> */}
+          <div className="space-y-2.5">
+            <div className="text-sm text-white">
+              {highest_risk?.message || subtitle}
+            </div>
+            {children}
           </div>
         </div>
       </div>
diff --git a/packages/nameguard-react/src/components/Search/SearchModal.tsx b/packages/nameguard-react/src/components/Search/SearchModal.tsx
index 45df99682..d712e2e46 100644
--- a/packages/nameguard-react/src/components/Search/SearchModal.tsx
+++ b/packages/nameguard-react/src/components/Search/SearchModal.tsx
@@ -21,7 +21,7 @@ export const SearchModal = () => {
 
   return (
     <Transition.Root show={modalOpen} as={Fragment}>
-      <Dialog as="div" className="relative z-20" onClose={handleClose}>
+      <Dialog as="div" className="relative z-40" onClose={handleClose}>
         <Transition.Child
           as={Fragment}
           enter="ease-out duration-300"
diff --git a/packages/nameguard-react/src/components/Shield/Shield.tsx b/packages/nameguard-react/src/components/Shield/Shield.tsx
index e341aea0a..5b6b655ca 100644
--- a/packages/nameguard-react/src/components/Shield/Shield.tsx
+++ b/packages/nameguard-react/src/components/Shield/Shield.tsx
@@ -64,7 +64,8 @@ export const Shield = ({ name }: ShieldProps) => {
           <div className="flex items-center justify-between">
             <span className={textClass}>{result.title}</span>
             <span className="text-sm font-normal text-gray-400">
-              {result.risk_count} risk{result.risk_count >= 0 && "s"} detected
+              {result.risk_count} risk{result.risk_count === 1 ? "" : "s"}{" "}
+              detected
             </span>
           </div>
           <div className="text-sm text-white">{result.subtitle}</div>