From 8c0130912821d0b11d4c1b37c18ac5104cb23c1f Mon Sep 17 00:00:00 2001
From: curben <2809763-curben@users.noreply.gitlab.com>
Date: Mon, 12 Aug 2019 13:57:03 +0930
Subject: [PATCH] fix(css): improve appearance of links

- remove underline when hover, makes hover more visible
- remove underline from link icon
- fill link icon with link's colour (inspired by MDN)
---
 themes/typing/source/css/typing.css | 27 ++++++++++++++++++---------
 1 file changed, 18 insertions(+), 9 deletions(-)

diff --git a/themes/typing/source/css/typing.css b/themes/typing/source/css/typing.css
index bc7a5f6..1091482 100644
--- a/themes/typing/source/css/typing.css
+++ b/themes/typing/source/css/typing.css
@@ -154,15 +154,6 @@ q:before {
   vertical-align: baseline;
 }
 
-.typo a {
-  border-bottom: 1px solid #1abc9c;
-}
-
-.typo a:hover {
-  border-bottom-color: #555;
-  color: #555;
-}
-
 .typo a:hover,
 a,
 ins {
@@ -908,6 +899,14 @@ blockquote {
   word-break: break-word;
 }
 
+.article .article-entry a {
+  border-bottom: 1px solid #1abc9c;
+}
+
+.article .article-entry a:hover {
+  border-bottom: none;
+}
+
 .article .article-entry img {
   display: block;
   margin: auto;
@@ -1398,6 +1397,16 @@ svg {
   border-bottom: none;
 }
 
+/* fill the link icon with link's color */
+svg#link {
+  fill: #1abc9c;
+}
+
+/* don't underline link icon */
+.article .article-entry a.headerlink {
+  border-bottom: none;
+}
+
 @media (max-width: 768px) {
   body {
     font-size: 1.2em;