diff mbox

[07/51] style.css: Tune <a> styling

Message ID 1440440620-25937-8-git-send-email-damien.lespiau@intel.com
State Changes Requested
Headers show

Commit Message

Damien Lespiau Aug. 24, 2015, 6:22 p.m. UTC 
From: Belén Barros Peña <belen.barros.pena@intel.com>

The links aren't underlined anymore. If, once visited, they are to
become black, we won't be able to distinguish them from regular text.

Instead let's decide about a color for links so they can be spotted,
even after a visit.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
---
 htdocs/css/style.css | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

Comments

Stephen Finucane Sept. 9, 2015, 1:56 p.m. UTC | #1 
Is there any reason we couldn't use the default Bootstrap-provided colours?
Stephen Finucane Sept. 9, 2015, 2:25 p.m. UTC | #2 
> No reason whatsoever. I do find it quite an intense link colour, though:
> if you have a lot of it, like in the patch table, it becomes quite tiring.
> I've attached a couple of screenshots: one with the default colour, the
> other with the suggested colour, so that you can see what I mean.
> 
> That was really my only reason to change it (well, that and keep it more
> in line with the current style).

Sounds fair.

Acked-by: Stephen Finucane <stephen.finucane@intel.com>
diff mbox

Patch

diff --git a/htdocs/css/style.css b/htdocs/css/style.css
index b66acc8..16a66d4 100644
--- a/htdocs/css/style.css
+++ b/htdocs/css/style.css
@@ -1,5 +1,10 @@ 
-a:visited { color: #000000; }
-a { color: #786fb4; }
+a {
+	color: #2E2E70;
+}
+
+a:hover {
+	color: #0A0A47;
+}
 
 .floaty {
 	position: fixed;