From 746de50ea85498830196fd6ad580364aa4691832 Mon Sep 17 00:00:00 2001 From: Deimos Date: Fri, 11 Jan 2019 18:07:06 -0700 Subject: [PATCH] Add underlines to links in posts I think this is a good idea for a few reasons, including accessibility (people that have difficulty distinguishing the link color will still be able to recognize links). --- tildes/scss/_placeholders.scss | 11 +++++++++++ tildes/scss/modules/_comment.scss | 1 + tildes/scss/modules/_message.scss | 1 + tildes/scss/modules/_topic.scss | 2 ++ 4 files changed, 15 insertions(+) diff --git a/tildes/scss/_placeholders.scss b/tildes/scss/_placeholders.scss index e28de29..8442a62 100644 --- a/tildes/scss/_placeholders.scss +++ b/tildes/scss/_placeholders.scss @@ -1,6 +1,17 @@ // Copyright (c) 2018 Tildes contributors // SPDX-License-Identifier: AGPL-3.0-or-later +// sets up links specifically for text content +%links-text { + a { + text-decoration: underline; + + &:hover { + text-decoration: none; + } + } +} + // sets up (nested) lists specifically for text content %lists-text { ol, ul { diff --git a/tildes/scss/modules/_comment.scss b/tildes/scss/modules/_comment.scss index 8af5047..dddea48 100644 --- a/tildes/scss/modules/_comment.scss +++ b/tildes/scss/modules/_comment.scss @@ -126,6 +126,7 @@ } .comment-text { + @extend %links-text; @extend %lists-text; padding: 0.2rem; diff --git a/tildes/scss/modules/_message.scss b/tildes/scss/modules/_message.scss index 0a2a3af..4a99ac7 100644 --- a/tildes/scss/modules/_message.scss +++ b/tildes/scss/modules/_message.scss @@ -24,6 +24,7 @@ } .message-text { + @extend %links-text; @extend %lists-text; margin-left: 0.2rem; diff --git a/tildes/scss/modules/_topic.scss b/tildes/scss/modules/_topic.scss index 7aa5ac2..929e433 100644 --- a/tildes/scss/modules/_topic.scss +++ b/tildes/scss/modules/_topic.scss @@ -147,6 +147,7 @@ } .topic-text-excerpt { + @extend %links-text; @extend %lists-text; display: none; @@ -232,6 +233,7 @@ } .topic-full-text { + @extend %links-text; @extend %lists-text; overflow: auto;