From 28356a9318a8a694040b3b847634f4c93cafe963 Mon Sep 17 00:00:00 2001
From: Cassie Tarakajian <ctarakajian@gmail.com>
Date: Thu, 31 Oct 2019 14:21:43 -0400
Subject: [PATCH] remove view column from asset list, add when clicking on
 asset row, opens asset in new tab

---
 client/modules/IDE/components/AssetList.jsx | 14 ++++++++------
 client/styles/components/_asset-list.scss   | 10 +++++++++-
 2 files changed, 17 insertions(+), 7 deletions(-)

diff --git a/client/modules/IDE/components/AssetList.jsx b/client/modules/IDE/components/AssetList.jsx
index a658fedf..3bec6cc2 100644
--- a/client/modules/IDE/components/AssetList.jsx
+++ b/client/modules/IDE/components/AssetList.jsx
@@ -56,19 +56,21 @@ class AssetList extends React.Component {
           <table className="asset-table">
             <thead>
               <tr>
-                <th>Name</th>
-                <th>Size</th>
-                <th>View</th>
-                <th>Sketch</th>
+                <th scope="col">Name</th>
+                <th scope="col">Size</th>
+                <th scope="col">Sketch</th>
               </tr>
             </thead>
             <tbody>
               {assetList.map(asset =>
                 (
                   <tr className="asset-table__row" key={asset.key}>
-                    <td>{asset.name}</td>
+                    <th scope="row">
+                      <Link to={asset.url} target="_blank">
+                        {asset.name}
+                      </Link>
+                    </th>
                     <td>{prettyBytes(asset.size)}</td>
-                    <td><Link to={asset.url} target="_blank">View</Link></td>
                     <td><Link to={`/${username}/sketches/${asset.sketchId}`}>{asset.sketchName}</Link></td>
                   </tr>
                 ))}
diff --git a/client/styles/components/_asset-list.scss b/client/styles/components/_asset-list.scss
index 1df44f20..654a8cee 100644
--- a/client/styles/components/_asset-list.scss
+++ b/client/styles/components/_asset-list.scss
@@ -21,7 +21,11 @@
     }
   }
 
-  & th {
+  .asset-table__row>th:nth-child(1) {
+    padding-left: #{12 / $base-font-size}rem;
+  }
+
+  & thead th {
     padding-top: #{10 / $base-font-size}rem;
     padding-bottom: #{15 / $base-font-size}rem;
     height: #{32 / $base-font-size}rem;
@@ -32,6 +36,10 @@
       background-color: getThemifyVariable('background-color');
     }
   }
+  
+  & th {
+    font-weight: normal;
+  }
 }
 
 .asset-table__row {