From 8f25d2905f69762bdd378d3d86a6352607f261bc Mon Sep 17 00:00:00 2001 From: Jenny <32821331+jenny-s51@users.noreply.github.com> Date: Wed, 4 Dec 2024 14:46:59 -0500 Subject: [PATCH] Theme Sidebar and Nav Items for Consistency with Central Dashboard UI (#596) Signed-off-by: Jenny <32821331+jenny-s51@users.noreply.github.com> remove unused components apply font color and border to selected nav items remove unused vars Signed-off-by: Jenny <32821331+jenny-s51@users.noreply.github.com> format and remove unused imports Signed-off-by: Jenny <32821331+jenny-s51@users.noreply.github.com> fix a11y issues Signed-off-by: Jenny <32821331+jenny-s51@users.noreply.github.com> add navbar Signed-off-by: Jenny <32821331+jenny-s51@users.noreply.github.com> temporarily revert fix to force a11y error in GH actions Signed-off-by: Jenny <32821331+jenny-s51@users.noreply.github.com> revert color back to fix a11y error Signed-off-by: Jenny <32821331+jenny-s51@users.noreply.github.com> remove hover color on brand image --- clients/ui/frontend/src/app/App.tsx | 21 +----- clients/ui/frontend/src/app/NavSidebar.tsx | 8 +++ clients/ui/frontend/src/images/logo.svg | 18 ++--- .../frontend/src/shared/style/MUI-theme.scss | 70 ++++++++++++++++++- 4 files changed, 87 insertions(+), 30 deletions(-) diff --git a/clients/ui/frontend/src/app/App.tsx b/clients/ui/frontend/src/app/App.tsx index 36c5e662..d4c7f091 100644 --- a/clients/ui/frontend/src/app/App.tsx +++ b/clients/ui/frontend/src/app/App.tsx @@ -3,22 +3,17 @@ import '@patternfly/react-core/dist/styles/base.css'; import './app.css'; import { Alert, - Brand, Bullseye, Button, Masthead, - MastheadBrand, MastheadContent, MastheadMain, - MastheadToggle, Page, PageSection, - PageToggleButton, Spinner, Stack, StackItem, } from '@patternfly/react-core'; -import { BarsIcon } from '@patternfly/react-icons'; import ToastNotifications from '~/shared/components/ToastNotifications'; import { useSettings } from '~/shared/hooks/useSettings'; import { isMUITheme, Theme } from '~/shared/utilities/const'; @@ -89,21 +84,7 @@ const App: React.FC = () => { const masthead = ( - - - - - - - - - - - + {/* TODO: [Auth-enablement] Add logout and user status once we enable itNavigates to register page from table toolbar */} diff --git a/clients/ui/frontend/src/app/NavSidebar.tsx b/clients/ui/frontend/src/app/NavSidebar.tsx index bfa9ab1a..80ed9c43 100644 --- a/clients/ui/frontend/src/app/NavSidebar.tsx +++ b/clients/ui/frontend/src/app/NavSidebar.tsx @@ -1,6 +1,7 @@ import * as React from 'react'; import { NavLink } from 'react-router-dom'; import { + Brand, Nav, NavExpandable, NavItem, @@ -46,6 +47,13 @@ const NavSidebar: React.FC = () => {