From 2243276c6b5c295efb6ca02b55ae321b9f7205ac Mon Sep 17 00:00:00 2001
From: Wesley Bomar <wbomar@tacc.utexas.edu>
Date: Mon, 20 Jun 2022 14:53:11 -0500
Subject: [PATCH] fix(core-components): Sidebar styles

1. Remove unused class "nav-content".
2. Use anchor tag pseduo classes to overwrite "elements.css".
3. Add missing style for nav content.

Depends on: e859114 (i.e. previous commit)
---
 libs/core-components/src/lib/Sidebar/Sidebar.jsx      |  2 +-
 .../src/lib/Sidebar/Sidebar.module.css                | 11 +++++++----
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/libs/core-components/src/lib/Sidebar/Sidebar.jsx b/libs/core-components/src/lib/Sidebar/Sidebar.jsx
index 499e92587..cfd98506d 100644
--- a/libs/core-components/src/lib/Sidebar/Sidebar.jsx
+++ b/libs/core-components/src/lib/Sidebar/Sidebar.jsx
@@ -25,7 +25,7 @@ const SidebarItem = ({ to, iconName, label, children, disabled, hidden }) => {
         <div
           className={`${disabled ? styles['disabled'] : ''} ${
             styles['content']
-          } nav-content`}
+          }`}
         >
           <Icon name={iconName} />
           <span className={styles['text']}>{label}</span>
diff --git a/libs/core-components/src/lib/Sidebar/Sidebar.module.css b/libs/core-components/src/lib/Sidebar/Sidebar.module.css
index 3109b2994..012078ed0 100644
--- a/libs/core-components/src/lib/Sidebar/Sidebar.module.css
+++ b/libs/core-components/src/lib/Sidebar/Sidebar.module.css
@@ -12,22 +12,25 @@
   padding-top: 20px;
 }
 
-.link {
+.link:is(:link, :visited) {
   color: var(--global-color-primary--dark);
   font-weight: 500;
 }
 .link:hover,
-.link--active {
+.link--active:is(:link, :visited) {
   color: var(--global-color-primary--x-dark); /* to pass color contrast test */
   background-color: var(--global-color-accent--weak);
 }
-.link--active,
-.link:active {
+.link:active,
+.link--active:is(:link, :visited) {
   font-weight: 700;
 }
 
 .content {
   padding-left: 20px;
+
+  display: flex;
+  align-items: center;
 }
 
 .text {