diff --git a/src/pages/sidepanel/State.tsx b/src/pages/sidepanel/State.tsx index d728f16..ad3e4e7 100644 --- a/src/pages/sidepanel/State.tsx +++ b/src/pages/sidepanel/State.tsx @@ -77,7 +77,7 @@ const State: React.FC> = () => { const handleStateToggle = isActive => { setResetSidePanel(Date.now()); extensionStateStorage.set(isActive); - if (!isActive) { + if (isActive) { chrome.tabs.query({ active: true, currentWindow: true }, tabs => { if (tabs.length > 0) { chrome.tabs.reload(tabs[0].id); diff --git a/src/pages/sidepanel/components/SectionHeader.tsx b/src/pages/sidepanel/components/SectionHeader.tsx index e48e06f..f532ec9 100644 --- a/src/pages/sidepanel/components/SectionHeader.tsx +++ b/src/pages/sidepanel/components/SectionHeader.tsx @@ -13,7 +13,7 @@ const HeaderContainer = styled(Box)(({ theme }) => ({ alignItems: 'center', gap: theme.spacing(1.25), // 10px between icon and text height: theme.spacing(5), // 40px - padding: `${theme.spacing(1.375)} ${theme.spacing(2)} ${theme.spacing(0.875)} ${theme.spacing(3)}`, // 11px 16px 7px 24px (top right bottom left) + padding: `1.375rem ${theme.spacing(2)} ${theme.spacing(0.875)} ${theme.spacing(3)}`, // 11px 16px 7px 24px (top right bottom left) width: '100%', opacity: 1, })); diff --git a/src/pages/sidepanel/components/TabNavigation.tsx b/src/pages/sidepanel/components/TabNavigation.tsx index c52b541..9f31c2b 100644 --- a/src/pages/sidepanel/components/TabNavigation.tsx +++ b/src/pages/sidepanel/components/TabNavigation.tsx @@ -17,6 +17,7 @@ interface TabNavigationProps { const TabsContainer = styled(Box)(({ theme }) => ({ padding: theme.spacing(2), + paddingTop: '1.5rem', // 24px })); const StyledTabs = styled(Tabs)(({ theme }) => ({ diff --git a/src/pages/sidepanel/components/TopNavigation.test.tsx b/src/pages/sidepanel/components/TopNavigation.test.tsx index 0534839..a29f5b9 100644 --- a/src/pages/sidepanel/components/TopNavigation.test.tsx +++ b/src/pages/sidepanel/components/TopNavigation.test.tsx @@ -81,12 +81,12 @@ describe('TopNavigation', () => { expect(screen.getByText('Disable Extension')).toBeInTheDocument(); }); - it('should render help and settings icons', () => { - renderWithTheme(); + // it('should render help and settings icons', () => { + // renderWithTheme(); - expect(screen.getByLabelText('help')).toBeInTheDocument(); - expect(screen.getByLabelText('settings')).toBeInTheDocument(); - }); + // expect(screen.getByLabelText('help')).toBeInTheDocument(); + // expect(screen.getByLabelText('settings')).toBeInTheDocument(); + // }); }); describe('Toggle Functionality', () => { @@ -123,14 +123,14 @@ describe('TopNavigation', () => { }); }); - describe('Settings Icon', () => { - it('should render settings icon with correct aria-label', () => { - renderWithTheme(); + // describe('Settings Icon', () => { + // it('should render settings icon with correct aria-label', () => { + // renderWithTheme(); - const settingsButton = screen.getByLabelText('settings'); - expect(settingsButton).toBeInTheDocument(); - }); - }); + // const settingsButton = screen.getByLabelText('settings'); + // expect(settingsButton).toBeInTheDocument(); + // }); + // }); describe('Auto Detection Indicator', () => { it('should call useAutoDetection with correct parameters', () => { diff --git a/src/pages/sidepanel/components/TopNavigation.tsx b/src/pages/sidepanel/components/TopNavigation.tsx index a6c7843..f6c9b01 100644 --- a/src/pages/sidepanel/components/TopNavigation.tsx +++ b/src/pages/sidepanel/components/TopNavigation.tsx @@ -10,7 +10,8 @@ import { IconButton, Divider, } from '@mui/material'; -import { HelpOutline, Settings } from '@mui/icons-material'; +// import { Settings } from '@mui/icons-material'; +import { HelpOutline } from '@mui/icons-material'; import Toggle from '@root/src/pages/sidepanel/components/Toggle'; import { AutoDetectionIndicator } from '@root/src/pages/sidepanel/components/AutoDetectionIndicator'; import { useAutoDetection } from '@root/src/pages/sidepanel/hooks/useAutoDetection'; @@ -101,7 +102,7 @@ const TopNavigation: React.FC = ({ isEnabled, onChange, domainState - = ({ isEnabled, onChange, domainState }} aria-label="settings"> - + */} diff --git a/src/pages/sidepanel/index.tsx b/src/pages/sidepanel/index.tsx index cc95fc6..15a5856 100644 --- a/src/pages/sidepanel/index.tsx +++ b/src/pages/sidepanel/index.tsx @@ -6,7 +6,7 @@ import { BrowserRouter as Router } from 'react-router-dom'; import State from '@pages/sidepanel/State'; import { AppThemeProvider } from '@src/theme/ThemeProvider'; import { QueryClient, QueryClientProvider } from '@tanstack/react-query'; -import { ReactQueryDevtools } from '@tanstack/react-query-devtools'; +// import { ReactQueryDevtools } from '@tanstack/react-query-devtools'; function init() { const appContainer = document.querySelector('#app-container'); @@ -25,7 +25,7 @@ function init() { - + {/* */} , diff --git a/src/pages/sidepanel/sections/profile/Attributes.tsx b/src/pages/sidepanel/sections/profile/Attributes.tsx index 20dca0d..aa82b86 100644 --- a/src/pages/sidepanel/sections/profile/Attributes.tsx +++ b/src/pages/sidepanel/sections/profile/Attributes.tsx @@ -59,10 +59,12 @@ const DescriptionText = styled(Typography)(() => ({ fontSize: appColors.common.fontSize.small, color: appColors.neutral[300], lineHeight: appColors.common.lineHeight.tight, + fontWeight: appColors.common.fontWeight.medium, })); const StyledLink = styled(Link)(() => ({ - font: `${appColors.common.fontWeight.bold} 0.75rem/1.42 Inter, sans-serif`, // Bold 12px, line-height 142% + fontWeight: appColors.common.fontWeight.bold, + fontSize: appColors.common.fontSize.small, color: appColors.common.colors.accent, textDecoration: 'underline', cursor: 'pointer', diff --git a/src/pages/sidepanel/sections/profile/Interests.tsx b/src/pages/sidepanel/sections/profile/Interests.tsx index 11c021a..b120c73 100644 --- a/src/pages/sidepanel/sections/profile/Interests.tsx +++ b/src/pages/sidepanel/sections/profile/Interests.tsx @@ -58,7 +58,8 @@ const EmptyStateText = styled(Typography)(() => ({ })); const StyledLink = styled(Link)(() => ({ - font: `${appColors.common.fontWeight.bold} 0.75rem/1.42 Inter, sans-serif`, // Bold 12px, line-height 142% + fontWeight: appColors.common.fontWeight.bold, + fontSize: appColors.common.fontSize.small, color: appColors.common.colors.accent, textDecoration: 'underline', cursor: 'pointer', @@ -71,6 +72,7 @@ const ContentText = styled(Typography)(() => ({ fontSize: appColors.common.fontSize.baseSmall, color: appColors.neutral[600], lineHeight: appColors.common.lineHeight.tight, + fontWeight: appColors.common.fontWeight.medium, })); export const Interests = ({