From 665b77fb4ee60e7662028a6a36ba7a65e11d8102 Mon Sep 17 00:00:00 2001
From: Niels Simenon <niels.simenon@gmail.com>
Date: Thu, 9 May 2019 01:58:47 +0200
Subject: [PATCH] Improved scene tile layout.

---
 .eslintrc             |  1 +
 assets/views/home.jsx | 50 +++++++++++++++++++------------------------
 public/css/style.css  | 47 ++++++++++++++++++++++++++++------------
 3 files changed, 56 insertions(+), 42 deletions(-)

diff --git a/.eslintrc b/.eslintrc
index b4941ed3..988dbb89 100644
--- a/.eslintrc
+++ b/.eslintrc
@@ -17,5 +17,6 @@
         "max-len": [2, {"code": 200, "tabWidth": 4, "ignoreUrls": true}],
         "react/jsx-uses-vars": 2,
         "react/jsx-indent": ["error", 4],
+        "react/jsx-indent-props": ["error", 4],
     }
 }
diff --git a/assets/views/home.jsx b/assets/views/home.jsx
index 6b308b60..5dc00708 100644
--- a/assets/views/home.jsx
+++ b/assets/views/home.jsx
@@ -12,15 +12,29 @@ class Home extends React.Component {
                 <ul className="scenes">
                     {this.props.releases.map(release => (
                         <li key={release.id} className="scene">
-                            <a
-                                href={`/item/${release.id}`}
-                                target="_blank"
+                            <span
+                                className="scene-banner"
                             >
+                                <span className="scene-details">
+                                    <a
+                                        href={`/site/${release.site.id}`}
+                                        className="scene-site site-link"
+                                        title={release.network.name}
+                                    >{release.site.name}</a>
+
+                                    <a
+                                        href={release.url}
+                                        target="_blank"
+                                        rel="noopener noreferrer"
+                                        className="scene-date"
+                                    >{moment(release.date).format('YYYY-MM-DD')}</a>
+                                </span>
+
                                 <img
                                     src={`/${release.site.id}/${release.id}/0.jpg`}
                                     className="scene-thumbnail"
                                 />
-                            </a>
+                            </span>
 
                             <div className="scene-info">
                                 <a
@@ -30,29 +44,6 @@ class Home extends React.Component {
                                     className="scene-row scene-link"
                                 ><h2 className="scene-title">{release.title}</h2></a>
 
-                                <span className="scene-row">
-                                    <a
-                                        href={`/site/${release.site.id}`}
-                                        className="scene-site site-link"
-                                    >{release.site.name}</a>
-
-                                    <span>
-                                        <a
-                                            className="scene-network"
-                                            href={`/network/${release.network.id}`}
-                                            target="_blank"
-                                            rel="noopener noreferrer"
-                                        >{release.network.name}</a>
-
-                                        <a
-                                            href={release.url}
-                                            target="_blank"
-                                            rel="noopener noreferrer"
-                                            className="scene-date"
-                                        >{moment(release.date).format('YYYY-MM-DD')}</a>
-                                    </span>
-                                </span>
-
                                 <span className="scene-row">
                                     <ul className="scene-actors nolist">{release.actors.map(actor =>
                                         <li
@@ -67,7 +58,10 @@ class Home extends React.Component {
                                     )}</ul>
                                 </span>
 
-                                <span className="scene-row">
+                                <span
+                                    title={release.tags.map(tag => tag.tag).join(', ')}
+                                    className="scene-row"
+                                >
                                     <ul className="scene-tags nolist">{release.tags.map(tag =>
                                         <li
                                             key={tag.tag}
diff --git a/public/css/style.css b/public/css/style.css
index f9edc82d..65cf2642 100644
--- a/public/css/style.css
+++ b/public/css/style.css
@@ -14,7 +14,7 @@ body {
 
 .scenes {
     display: grid;
-    grid-template-columns: repeat(auto-fit, minmax(20rem, 1fr));
+    grid-template-columns: repeat(auto-fit, minmax(18rem, 1fr));
     list-style: none;
     padding: 0;
     margin: 0;
@@ -24,11 +24,23 @@ body {
     display: flex;
     flex-direction: column;
     box-sizing: border-box;
+    padding: 0 0 .5rem 0;;
     margin: .5rem;
-    height: 22rem;
     box-shadow: 0 0 3px rgba(0, 0, 0, .5);
 }
 
+.scene-banner {
+    position: relative;
+}
+
+.scene-details {
+    width: 100%;
+    display: flex;
+    align-items: center;
+    justify-content: space-between;
+    position: absolute;
+}
+
 .scene-thumbnail {
     width: 100%;
     height: 200px;
@@ -40,7 +52,9 @@ body {
 .scene-row {
     display: flex;
     justify-content: space-between;
-    padding: .25rem .5rem;
+    align-items: center;
+    box-sizing: border-box;
+    padding: .1rem .5rem;
 }
 
 .scene-info {
@@ -55,16 +69,19 @@ body {
     color: #000;
     margin: 0;
     font-size: 1rem;
+    word-wrap: break-word;
+    overflow: hidden;
+    max-height: 3rem;
+    line-height: 1.5rem;
 }
 
-.scene-site {
-    font-weight: bold;
-    font-size: .8rem;
-}
-
+.scene-site,
 .scene-date {
-    color: #555;
+    color: #fff;
+    background: rgba(0, 0, 0, .5);
     font-size: .8rem;
+    padding: .25rem;
+    text-decoration: none;
 }
 
 .scene-network {
@@ -74,8 +91,10 @@ body {
 }
 
 .scene-tags {
-    white-space: nowrap;
+    word-wrap: break-word;
     overflow: hidden;
+    max-height: 2.5rem;
+    line-height: 1.25rem;
 }
 
 .scene-actor,
@@ -83,6 +102,10 @@ body {
     margin: 0 .25rem 0 0;
 }
 
+.scene-actor {
+    font-size: .9rem;
+}
+
 .scene-tag {
     font-size: .75rem;
 }
@@ -92,10 +115,6 @@ body {
     content: ",";
 }
 
-.site-link {
-    color: #000;
-}
-
 .actor-link,
 .tag-link {
     color: #000;