From 96897be941795908a312c71b095214c0f0894420 Mon Sep 17 00:00:00 2001
From: Jimmy Cai <jimmehcai@gmail.com>
Date: Thu, 24 Sep 2020 23:59:00 +0200
Subject: [PATCH] feat(article list): article clickable and equal height

---
 assets/scss/partials/article.scss          | 28 +++++++++----
 assets/scss/partials/layout/article.scss   |  2 +-
 assets/scss/variables.scss                 |  5 ++-
 layouts/partials/article-list/compact.html | 46 +++++++++++-----------
 4 files changed, 48 insertions(+), 33 deletions(-)

diff --git a/assets/scss/partials/article.scss b/assets/scss/partials/article.scss
index 834912d..0a22ef7 100644
--- a/assets/scss/partials/article.scss
+++ b/assets/scss/partials/article.scss
@@ -11,8 +11,8 @@
         border-radius: var(--card-border-radius);
         overflow: hidden;
 
-        transition: box-shadow .3s ease;
-        
+        transition: box-shadow 0.3s ease;
+
         &:hover {
             box-shadow: var(--shadow-l2);
         }
@@ -159,24 +159,32 @@
     border-radius: var(--card-border-radius);
     box-shadow: var(--shadow-l1);
     background-color: var(--card-background);
+    --image-size: 60px;
+
+    @media (max-width: $on-tablet) {
+        --image-size: 50px;
+    }
 
     & + .pagination {
         margin-top: var(--section-separation);
     }
 
     article {
-        display: flex;
-        align-items: center;
-        padding: var(--small-card-padding);
+        & > a {
+            display: flex;
+            align-items: center;
+            padding: var(--small-card-padding);
+        }
 
         &:not(:last-of-type) {
-            border-bottom: 2px solid var(--card-separator-color);
+            border-bottom: 1.5px solid var(--card-separator-color);
         }
 
         .article-details {
             flex-grow: 1;
             padding: 0;
             padding-right: 15px;
+            min-height: var(--image-size);
         }
 
         .article-title {
@@ -190,10 +198,14 @@
 
         .article-image {
             img {
-                width: 60px;
-                height: 60px;
+                width: var(--image-size);
+                height: var(--image-size);
             }
         }
+
+        .article-time {
+            font-size: 1.4rem;
+        }
     }
 }
 
diff --git a/assets/scss/partials/layout/article.scss b/assets/scss/partials/layout/article.scss
index cd36496..5427859 100644
--- a/assets/scss/partials/layout/article.scss
+++ b/assets/scss/partials/layout/article.scss
@@ -43,7 +43,7 @@
         }
     }
 
-    article {
+    .main-article {
         background: var(--card-background);
         border-radius: var(--card-border-radius);
         box-shadow: var(--shadow-l1);
diff --git a/assets/scss/variables.scss b/assets/scss/variables.scss
index 8c40575..d3ed0ce 100644
--- a/assets/scss/variables.scss
+++ b/assets/scss/variables.scss
@@ -63,6 +63,9 @@ $defaultTagColors: #fff, #fff, #fff, #fff, #fff;
     }
 
     --small-card-padding: 25px;
+    @media (max-width: $on-tablet) {
+        --small-card-padding: 25px 20px;
+    }
 
     @media (prefers-color-scheme: dark) {
         --card-background: #424242;
@@ -129,4 +132,4 @@ $defaultTagColors: #fff, #fff, #fff, #fff, #fff;
     --shadow-l3: 0px 10px 20px rgba(0, 0, 0, 0.04), 0px 2px 6px rgba(0, 0, 0, 0.04), 0px 0px 1px rgba(0, 0, 0, 0.04);
     --shadow-l4: 0px 24px 32px rgba(0, 0, 0, 0.04), 0px 16px 24px rgba(0, 0, 0, 0.04), 0px 4px 8px rgba(0, 0, 0, 0.04),
         0px 0px 1px rgba(0, 0, 0, 0.04);
-}
\ No newline at end of file
+}
diff --git a/layouts/partials/article-list/compact.html b/layouts/partials/article-list/compact.html
index e9fa77d..ddef710 100644
--- a/layouts/partials/article-list/compact.html
+++ b/layouts/partials/article-list/compact.html
@@ -1,27 +1,27 @@
 <article>
-    <div class="article-details">
-        <h2 class="article-title">
-            <a href="{{ .Permalink }}">
+    <a href="{{ .Permalink }}">
+        <div class="article-details">
+            <h2 class="article-title">
                 {{- .Title -}}
-            </a>
-        </h2>
-        <footer class="article-time">
-            <time datetime='{{ .Date.Format "2006-01-02T15:04:05Z07:00" }}'>
-                {{- .Date.Format (or .Site.Params.dateFormat.published "Jan 02, 2006") -}}
-            </time>
-        </footer>
-    </div>
-
-    {{- $image := partialCached "helper/image" (dict "Context" . "Type" "articleList") .RelPermalink "articleList" -}}
-    {{ if $image.exists }}
-        <div class="article-image">
-            {{ if $image.resource }}
-                {{- $thumbnail := $image.resource.Fill "120x120" -}}
-                <img src="{{ $thumbnail.RelPermalink }}" width="{{ $thumbnail.Width }}"
-                    height="{{ $thumbnail.Height }}" loading="lazy">
-            {{ else }}
-                <img src="{{ $image.permalink }}" loading="lazy" alt="Featured image of post {{ .Title }}" />
-            {{ end }}
+            </h2>
+            <footer class="article-time">
+                <time datetime='{{ .Date.Format "2006-01-02T15:04:05Z07:00" }}'>
+                    {{- .Date.Format (or .Site.Params.dateFormat.published "Jan 02, 2006") -}}
+                </time>
+            </footer>
         </div>
-    {{ end }}
+
+        {{- $image := partialCached "helper/image" (dict "Context" . "Type" "articleList") .RelPermalink "articleList" -}}
+        {{ if $image.exists }}
+            <div class="article-image">
+                {{ if $image.resource }}
+                    {{- $thumbnail := $image.resource.Fill "120x120" -}}
+                    <img src="{{ $thumbnail.RelPermalink }}" width="{{ $thumbnail.Width }}"
+                        height="{{ $thumbnail.Height }}" loading="lazy">
+                {{ else }}
+                    <img src="{{ $image.permalink }}" loading="lazy" alt="Featured image of post {{ .Title }}" />
+                {{ end }}
+            </div>
+        {{ end }}
+    </a>
 </article>
\ No newline at end of file