From 1172d9cd41589711c182542a0f9f9fe5507d42d3 Mon Sep 17 00:00:00 2001
From: Justin Starry <justin@solana.com>
Date: Wed, 22 Apr 2020 22:01:56 +0800
Subject: [PATCH] Make more things copyable

---
 explorer/src/components/AccountModal.tsx     |  9 ++++--
 explorer/src/components/AccountsCard.tsx     | 10 +++++-
 explorer/src/components/Copyable.tsx         | 34 +++++++++++++-------
 explorer/src/components/TransactionModal.tsx | 25 +++++++++++---
 explorer/src/scss/_solana.scss               | 16 ++++++---
 5 files changed, 70 insertions(+), 24 deletions(-)

diff --git a/explorer/src/components/AccountModal.tsx b/explorer/src/components/AccountModal.tsx
index 0608e8db23..9a8fedf311 100644
--- a/explorer/src/components/AccountModal.tsx
+++ b/explorer/src/components/AccountModal.tsx
@@ -7,6 +7,7 @@ import {
   Status
 } from "../providers/accounts";
 import { TransactionError } from "@solana/web3.js";
+import Copyable from "./Copyable";
 
 function AccountModal() {
   const selected = useSelectedAccount();
@@ -135,9 +136,11 @@ function ListGroupItem({
           </span>
         </div>
         <div className="col min-width-0">
-          <h5 className="mb-0 text-truncate">
-            <code>{signature}</code>
-          </h5>
+          <Copyable text={signature}>
+            <h5 className="mb-0 text-truncate">
+              <code>{signature}</code>
+            </h5>
+          </Copyable>
         </div>
       </div>
     </div>
diff --git a/explorer/src/components/AccountsCard.tsx b/explorer/src/components/AccountsCard.tsx
index 7427d9f4e4..a0a015c8e2 100644
--- a/explorer/src/components/AccountsCard.tsx
+++ b/explorer/src/components/AccountsCard.tsx
@@ -209,7 +209,15 @@ const renderAccountRow = (
       </td>
       <td>{balance}</td>
       <td>{data}</td>
-      <td>{owner === "-" ? owner : <code>{owner}</code>}</td>
+      <td>
+        {owner === "-" ? (
+          owner
+        ) : (
+          <Copyable text={owner}>
+            <code>{owner}</code>
+          </Copyable>
+        )}
+      </td>
       <td>{renderDetails()}</td>
     </tr>
   );
diff --git a/explorer/src/components/Copyable.tsx b/explorer/src/components/Copyable.tsx
index 5c1d790e0c..594afceb05 100644
--- a/explorer/src/components/Copyable.tsx
+++ b/explorer/src/components/Copyable.tsx
@@ -5,27 +5,39 @@ type CopyableProps = {
   children: ReactNode;
 };
 
-const popover = (
-  <div className="popover fade bs-popover-right show">
-    <div className="arrow" />
-    <div className="popover-body">Copied!</div>
-  </div>
-);
+type State = "hide" | "copy" | "copied";
+
+function Popover({ state }: { state: State }) {
+  if (state === "hide") return null;
+  const text = state === "copy" ? "Copy" : "Copied!";
+  return (
+    <div className="popover bs-popover-top show">
+      <div className="arrow" />
+      <div className="popover-body">{text}</div>
+    </div>
+  );
+}
 
 function Copyable({ text, children }: CopyableProps) {
-  const [showPopover, setShowPopover] = useState(false);
+  const [state, setState] = useState<State>("hide");
 
   const copyToClipboard = () => navigator.clipboard.writeText(text);
   const handleClick = () =>
     copyToClipboard().then(() => {
-      setShowPopover(true);
-      setTimeout(setShowPopover.bind(null, false), 2500);
+      setState("copied");
+      setTimeout(() => setState("hide"), 1000);
     });
 
   return (
     <div className="copyable">
-      <div onClick={handleClick}>{children}</div>
-      {showPopover && popover}
+      <div
+        onClick={handleClick}
+        onMouseOver={() => setState("copy")}
+        onMouseOut={() => state === "copy" && setState("hide")}
+      >
+        {children}
+      </div>
+      <Popover state={state} />
     </div>
   );
 }
diff --git a/explorer/src/components/TransactionModal.tsx b/explorer/src/components/TransactionModal.tsx
index b870006e23..78cfccad37 100644
--- a/explorer/src/components/TransactionModal.tsx
+++ b/explorer/src/components/TransactionModal.tsx
@@ -13,6 +13,7 @@ import {
   CreateAccountParams,
   TransactionInstruction
 } from "@solana/web3.js";
+import Copyable from "./Copyable";
 
 function TransactionModal() {
   const { selected } = useTransactions();
@@ -106,15 +107,21 @@ function TransferDetails({
   transfer: TransferParams;
   index: number;
 }) {
+  const from = transfer.fromPubkey.toBase58();
+  const to = transfer.toPubkey.toBase58();
   return (
     <div className="card-body">
       <h4 className="ix-pill">{`Instruction #${index + 1} (Transfer)`}</h4>
       <div className="list-group list-group-flush my-n3">
         <ListGroupItem label="From">
-          <code>{transfer.fromPubkey.toBase58()}</code>
+          <Copyable text={from}>
+            <code>{from}</code>
+          </Copyable>
         </ListGroupItem>
         <ListGroupItem label="To">
-          <code>{transfer.toPubkey.toBase58()}</code>
+          <Copyable text={to}>
+            <code>{to}</code>
+          </Copyable>
         </ListGroupItem>
         <ListGroupItem label="Amount (SOL)">
           {`◎${(1.0 * transfer.lamports) / LAMPORTS_PER_SOL}`}
@@ -131,16 +138,22 @@ function CreateDetails({
   create: CreateAccountParams;
   index: number;
 }) {
+  const from = create.fromPubkey.toBase58();
+  const newKey = create.newAccountPubkey.toBase58();
   return (
     <div className="card-body">
       <h4 className="ix-pill">{`Instruction #${index +
         1} (Create Account)`}</h4>
       <div className="list-group list-group-flush my-n3">
         <ListGroupItem label="From">
-          <code>{create.fromPubkey.toBase58()}</code>
+          <Copyable text={from}>
+            <code>{from}</code>
+          </Copyable>
         </ListGroupItem>
         <ListGroupItem label="New Account">
-          <code>{create.newAccountPubkey.toBase58()}</code>
+          <Copyable text={newKey}>
+            <code>{newKey}</code>
+          </Copyable>
         </ListGroupItem>
         <ListGroupItem label="Amount (SOL)">
           {`◎${(1.0 * create.lamports) / LAMPORTS_PER_SOL}`}
@@ -167,7 +180,9 @@ function InstructionDetails({
       <div className="list-group list-group-flush my-n3">
         {ix.keys.map(({ pubkey }, keyIndex) => (
           <ListGroupItem key={keyIndex} label={`Address #${keyIndex + 1}`}>
-            <code>{pubkey.toBase58()}</code>
+            <Copyable text={pubkey.toBase58()}>
+              <code>{pubkey.toBase58()}</code>
+            </Copyable>
           </ListGroupItem>
         ))}
         <ListGroupItem label="Data (Bytes)">{ix.data.length}</ListGroupItem>
diff --git a/explorer/src/scss/_solana.scss b/explorer/src/scss/_solana.scss
index ac34b49f73..37091dcde5 100644
--- a/explorer/src/scss/_solana.scss
+++ b/explorer/src/scss/_solana.scss
@@ -17,10 +17,18 @@ code {
     cursor: pointer;
   }
 
-  .popover {
-    top: -1rem;
-    right: -5.12rem;
-    left: auto;
+  .popover.bs-popover-top {
+    background-color: $dark;
+    top: -4rem;
+    left: 40%;
+
+    .popover-body {
+      color: white;
+    }
+
+    .arrow::after {
+      border-top-color: $dark;
+    }
   }
 }