From 475b18d171efc5a1fec300ac632dae8e506b254a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Javi=20Mart=C3=ADn?= Date: Tue, 24 Aug 2021 12:50:20 +0200 Subject: [PATCH] Adjust hover styles in table actions The underline text decoration looked a bit strange with the icon on top of it, and we had an inconsistency since links had a text decoration but buttons didn't. I think we can remove it since both the color of the icon and the cursor change on hover and that's enough feedback. And we were accidentally overwriting the color of the investments link on hover and focus. --- app/assets/stylesheets/admin/table_actions.scss | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/assets/stylesheets/admin/table_actions.scss b/app/assets/stylesheets/admin/table_actions.scss index 726769977..f2e1775d6 100644 --- a/app/assets/stylesheets/admin/table_actions.scss +++ b/app/assets/stylesheets/admin/table_actions.scss @@ -19,6 +19,7 @@ &:hover, &:focus { color: $link-hover; + text-decoration: none; } &::before { @@ -77,7 +78,7 @@ .investments-link { color: darken($warning-color, 20%); - &::before { + &:not(:hover, :focus)::before { color: $warning-color; } }