From 2e7f53195d7a43b58e37f23bf9337392f2c72a31 Mon Sep 17 00:00:00 2001
From: Matthias Kretschmann <m@kretschmann.io>
Date: Sun, 14 Oct 2018 13:59:18 +0200
Subject: [PATCH] actions and spacing tweaks

---
 src/components/Layout.module.scss           |  2 +-
 src/components/Post/PostActions.jsx         | 23 +++----
 src/components/Post/PostActions.module.scss | 70 ++++++++++++++-------
 src/components/Post/PostMeta.module.scss    |  6 +-
 src/styles/_code.scss                       |  8 ++-
 src/styles/_syntax.scss                     |  8 ++-
 src/styles/_variables.scss                  |  8 ++-
 src/styles/global.scss                      |  5 +-
 8 files changed, 84 insertions(+), 46 deletions(-)

diff --git a/src/components/Layout.module.scss b/src/components/Layout.module.scss
index 41b5312b..d5cd1266 100644
--- a/src/components/Layout.module.scss
+++ b/src/components/Layout.module.scss
@@ -9,7 +9,7 @@
 }
 
 .content {
-    padding: 0 $spacer;
+    padding: 0 $spacer / $line-height;
     width: 100%;
 
     @media (min-width: $screen-sm) {
diff --git a/src/components/Post/PostActions.jsx b/src/components/Post/PostActions.jsx
index e6339aae..9df4d607 100644
--- a/src/components/Post/PostActions.jsx
+++ b/src/components/Post/PostActions.jsx
@@ -25,29 +25,24 @@ export default class PostActions extends PureComponent {
 
     return (
       <aside className={styles.actions}>
-        <article className={styles.action}>
+        <a
+          className={styles.action}
+          href={`https://twitter.com/intent/tweet?text=@kremalicious&url=${url}${slug}`}
+        >
           <Twitter />
           <h1 className={styles.actionTitle}>Have a comment?</h1>
           <p className={styles.actionText}>
-            Hit me up{' '}
-            <a
-              href={`https://twitter.com/intent/tweet?text=@kremalicious&url=${url}${slug}`}
-            >
-              @kremalicious
-            </a>
-            .
+            Hit me up <span className={styles.link}>@kremalicious</span>.
           </p>
-        </article>
-        <article className={styles.action}>
+        </a>
+        <button className={styles.action} onClick={this.toggleModal}>
           <Bitcoin />
           <h1 className={styles.actionTitle}>Found something useful?</h1>
           <p className={styles.actionText}>
             Say thanks{' '}
-            <button className="link" onClick={this.toggleModal}>
-              with Bitcoins or Ether.
-            </button>
+            <span className={styles.link}>with Bitcoins or Ether</span>.
           </p>
-        </article>
+        </button>
 
         {this.state.showModal && (
           <ModalThanks
diff --git a/src/components/Post/PostActions.module.scss b/src/components/Post/PostActions.module.scss
index 57e6587e..7079322e 100644
--- a/src/components/Post/PostActions.module.scss
+++ b/src/components/Post/PostActions.module.scss
@@ -1,12 +1,10 @@
 @import 'variables';
-@import 'mixins';
 
 .actions {
-    @include divider;
-    @include divider-top;
-
     margin-top: $spacer * 3;
-    padding: $spacer * $line-height 0;
+    padding: ($spacer * $line-height) 0 ($spacer * $line-height) 100%;
+    background: rgba(#fff, .5);
+    margin-left: -100%;
 
     @media (min-width: $screen-xs) {
         display: flex;
@@ -14,39 +12,69 @@
     }
 }
 
+.link {
+    transition: .2s ease-out;
+    color: $link-color;
+}
+
+.actionTitle {
+    font-size: $font-size-h4;
+    line-height: $line-height;
+    color: $text-color;
+    margin-top: 0;
+    margin-bottom: 0;
+    transition: color .2s ease-out;
+}
+
+.actionText {
+    font-size: $font-size-base;
+    color: $brand-grey-light;
+    margin-bottom: 0;
+    transition: color .2s ease-out;
+}
+
 .action {
+    display: block;
+    margin: 0;
     padding-left: 1.75rem;
+    padding-right: $spacer / 2;
     position: relative;
+    text-align: left;
 
     &:first-child {
-        margin-bottom: $spacer * 2;
+        margin-bottom: $spacer * $line-height;
     }
 
     @media (min-width: $screen-xs) {
-        flex: 1 1 48%;
+        flex: 1 1 50%;
 
         &:first-child {
             margin-bottom: 0;
         }
     }
 
+    &:hover,
+    &:focus {
+        .link,
+        .actionTitle,
+        .actionText {
+            color: $link-color-hover;
+        }
+    }
+
+    &:active {
+        .link,
+        .actionTitle,
+        .actionText {
+            transition: none;
+            color: $link-color-active;
+        }
+    }
+
     svg {
         position: absolute;
         left: 0;
-        top: .2rem;
+        top: .4rem;
         fill: $brand-grey-light;
     }
 }
-
-.actionTitle {
-    font-size: $font-size-base;
-    line-height: $line-height;
-    color: $text-color;
-    margin-top: 0;
-    margin-bottom: 0;
-}
-
-.actionText {
-    font-size: $font-size-small;
-    margin-bottom: 0;
-}
diff --git a/src/components/Post/PostMeta.module.scss b/src/components/Post/PostMeta.module.scss
index d82251d3..6c768906 100644
--- a/src/components/Post/PostMeta.module.scss
+++ b/src/components/Post/PostMeta.module.scss
@@ -4,8 +4,9 @@
 /////////////////////////////////////
 
 .entryMeta {
-    font-size: $font-size-small;
+    font-size: $font-size-base;
     margin-top: $spacer * 2;
+    color: $brand-grey-light;
 }
 
 .byline,
@@ -72,6 +73,9 @@
     color: $text-color;
     margin-left: $spacer / 2;
     margin-right: $spacer / 2;
+    margin-bottom: $spacer / 2;
+    white-space: nowrap;
+    display: inline-block;
 
     &::before {
         color: $brand-grey-light;
diff --git a/src/styles/_code.scss b/src/styles/_code.scss
index 4979db36..00379c31 100644
--- a/src/styles/_code.scss
+++ b/src/styles/_code.scss
@@ -38,8 +38,6 @@ pre {
     line-height: $line-height;
     color: $code-color;
     background-color: $code-bg;
-    margin-left: -($spacer);
-    margin-right: -($spacer);
     border-radius: $border-radius;
 
     // make 'em scrollable
@@ -49,6 +47,8 @@ pre {
 
     @media (min-width: $screen-sm) {
         max-height: 500px;
+        margin-left: -($spacer);
+        margin-right: -($spacer);
     }
 
     @media (min-width: $screen-md) {
@@ -71,6 +71,10 @@ pre {
         overflow-wrap: normal;
         word-wrap: normal;
         word-break: normal;
+
+        > span {
+            padding-right: $spacer;
+        }
     }
 }
 
diff --git a/src/styles/_syntax.scss b/src/styles/_syntax.scss
index 39f5da9b..43a67243 100644
--- a/src/styles/_syntax.scss
+++ b/src/styles/_syntax.scss
@@ -1,4 +1,5 @@
 /* stylelint-disable */
+@import 'variables';
 
 /*
 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
@@ -11,8 +12,11 @@ copyright  Copyright (C) 2017                                  +
 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 */
 .nord {
-    margin-left: -($spacer);
-    margin-right: -($spacer);
+    @media (min-width: $screen-sm) {
+        max-height: 500px;
+        margin-left: -($spacer);
+        margin-right: -($spacer);
+    }
 
     > pre {
         margin-left: 0;
diff --git a/src/styles/_variables.scss b/src/styles/_variables.scss
index abc21c60..670f9527 100644
--- a/src/styles/_variables.scss
+++ b/src/styles/_variables.scss
@@ -35,6 +35,8 @@ $text-color--dark: lighten($brand-grey-light, 5%);
 $text-color-light--dark: lighten($brand-grey-light, 5%);
 
 $link-color: $brand-cyan;
+$link-color-hover: lighten($link-color, 10%);
+$link-color-active: darken($link-color, 10%);
 
 // Base Typography
 /////////////////////////////////////
@@ -49,9 +51,9 @@ $font-size-mini: .7rem;
 $font-size-h1: 2.5rem;
 $font-size-h2: 2rem;
 $font-size-h3: 1.5rem;
-$font-size-h4: $font-size-large;
-$font-size-h5: $font-size-base;
-$font-size-h6: $font-size-small;
+$font-size-h4: 1.2rem;
+$font-size-h5: $font-size-large;
+$font-size-h6: $font-size-base;
 
 $line-height: 1.5;
 $line-height-small: 1.1428571429;
diff --git a/src/styles/global.scss b/src/styles/global.scss
index 691ccb34..b5887b56 100644
--- a/src/styles/global.scss
+++ b/src/styles/global.scss
@@ -74,6 +74,7 @@ button {
     background: transparent;
     border: 0;
     -webkit-appearance: none;
+    outline: 0;
 
     &:active {
         transition: none;
@@ -178,12 +179,12 @@ a {
     &:hover,
     &:focus {
         outline: 0;
-        color: lighten($link-color, 10%);
+        color: $link-color-hover;
     }
 
     &:active {
         transition: none;
-        color: darken($link-color, 10%);
+        color: $link-color-active;
     }
 }