From 6f7f32ef8718b8fc79732e1af84e2dfff63a7bac Mon Sep 17 00:00:00 2001
From: Itay Cohen <itaycohen23@gmail.com>
Date: Thu, 14 May 2020 18:45:29 +0300
Subject: [PATCH] Small formatting for the results table (#167)

* Results table formatting

* Adress feedback from review
---
 src/mqueryfront/src/App.css               | 17 +++++
 src/mqueryfront/src/QueryResultsStatus.js | 89 +++++++++++++----------
 2 files changed, 67 insertions(+), 39 deletions(-)

diff --git a/src/mqueryfront/src/App.css b/src/mqueryfront/src/App.css
index a9c85d7c..b97b9141 100644
--- a/src/mqueryfront/src/App.css
+++ b/src/mqueryfront/src/App.css
@@ -10,6 +10,23 @@
 
 .mquery-scroll-matches td {
     word-break: break-all;
+    font-family: "monospace"
+}
+
+.mquery-scroll-matches td i {
+    visibility: hidden;
+}
+
+.mquery-scroll-matches td:hover i {
+    visibility: visible;
+}
+
+.mquery-scroll-matches td i:hover {
+    color: var(--blue);
+}
+
+.copyable-item {
+    cursor: pointer;
 }
 
 .is-collapsed {
diff --git a/src/mqueryfront/src/QueryResultsStatus.js b/src/mqueryfront/src/QueryResultsStatus.js
index 85ec701e..8ec24702 100644
--- a/src/mqueryfront/src/QueryResultsStatus.js
+++ b/src/mqueryfront/src/QueryResultsStatus.js
@@ -8,6 +8,17 @@ import { CopyToClipboard } from "react-copy-to-clipboard";
 import ActionCancel from "./components/ActionCancel";
 
 function MatchItem(props) {
+    const download_url =
+        API_URL +
+        "/download?job_id=" +
+        encodeURIComponent(props.qhash) +
+        "&ordinal=" +
+        encodeURIComponent(props.ordinal) +
+        "&file_path=" +
+        encodeURIComponent(props.file);
+
+    const path = require("path");
+
     const metadata = Object.values(props.meta)
         .filter((v) => !v.hidden)
         .map((v) => (
@@ -19,20 +30,6 @@ function MatchItem(props) {
             </a>
         ));
 
-    let hashes = Object.values(props.meta)
-        .filter((v) => v.hidden)
-        .map((v) => (
-            <CopyToClipboard text={v.display_text} key={v}>
-                <div
-                    style={{ fontFamily: "monospace" }}
-                    data-toggle="tooltip"
-                    title="Click to copy"
-                >
-                    {v.display_text}
-                </div>
-            </CopyToClipboard>
-        ));
-
     let matches = <span></span>;
     if (props.matches) {
         matches = Object.values(props.matches).map((v) => (
@@ -44,35 +41,54 @@ function MatchItem(props) {
         ));
     }
 
-    const download_url =
-        API_URL +
-        "/download?job_id=" +
-        encodeURIComponent(props.qhash) +
-        "&ordinal=" +
-        encodeURIComponent(props.ordinal) +
-        "&file_path=" +
-        encodeURIComponent(props.file);
-
-    const path = require("path");
-
     return (
         <tr>
             <td>
                 <div className="row m-0 text-truncate">
                     <div className="text-truncate" style={{ minWidth: 50 }}>
-                        <a
-                            href={download_url}
-                            data-toggle="tooltip"
-                            title={props.file}
-                        >
+                        <div>
+                            {props.meta.sha256.display_text}
+                            <small>
+                                <a
+                                    href={download_url}
+                                    data-toggle="tooltip"
+                                    title={props.file}
+                                    className="text-secondary"
+                                >
+                                    <i className="fa fa-download fa-xm ml-2" />
+                                </a>
+                                <CopyToClipboard
+                                    text={props.meta.sha256.display_text}
+                                    className="copyable-item"
+                                >
+                                    <span
+                                        data-toggle="tooltip"
+                                        title="Copy sha256 to clipboard"
+                                    >
+                                        <i className="fa fa-copy fa-xm ml-2" />
+                                    </span>
+                                </CopyToClipboard>
+                            </small>
+                        </div>
+                        <small class="text-secondary">
                             {path.basename(props.file)}
-                        </a>
+                            <CopyToClipboard
+                                text={path.basename(props.file)}
+                                className="copyable-item"
+                            >
+                                <span
+                                    data-toggle="tooltip"
+                                    title="Copy file name to clipboard"
+                                >
+                                    <i className="fa fa-copy fa-xm ml-2" />
+                                </span>
+                            </CopyToClipboard>
+                        </small>
+                        {matches}
+                        {metadata}
                     </div>
-                    {matches}
-                    {metadata}
                 </div>
             </td>
-            {props.collapsed ? <td>{hashes}</td> : null}
         </tr>
     );
 }
@@ -219,11 +235,6 @@ class QueryResultsStatus extends Component {
                         <thead>
                             <tr>
                                 <th className="col-md-8">Matches</th>
-                                {this.props.collapsed && (
-                                    <th className="col-md-4 d-none d-sm-table-cell">
-                                        SHA256
-                                    </th>
-                                )}
                             </tr>
                         </thead>
                         <tbody>{matches}</tbody>