farmcontrol/farmcontrol-ui
farmcontrol/farmcontrol-ui
2
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

Fix variable naming inconsistency in DashboardNavigation component
All checks were successful
farmcontrol/farmcontrol-ui/pipeline/head This commit looks good
Details

Browse Source 
- Updated `isFullscreen` to `isFullScreen` for consistency with context naming conventions.
- Adjusted class logic in the navigation wrapper to reflect the updated variable name, ensuring proper styling application.
This commit is contained in:
Tom Butcher 2026-08-04 02:29:26 +01:00
parent 184a6621d9
commit 19ce749e80
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/components/Dashboard/common/DashboardNavigation.jsx
View File

@ -65,7 +65,7 @@ const DashboardNavigation = () => {
icon: <ProductionIcon /> icon: <ProductionIcon />
}) })
const isMobile = useMediaQuery({ maxWidth: 768 }) const isMobile = useMediaQuery({ maxWidth: 768 })
const { platform, isElectron, setSidebarViewMenu, isFullscreen } = const { platform, isElectron, setSidebarViewMenu, isFullScreen } =
useContext(ElectronContext) useContext(ElectronContext)
const { availableUpdate, checkForUpdates } = useAppUpdateContext() const { availableUpdate, checkForUpdates } = useAppUpdateContext()
const mainMenuItems = useMemo( const mainMenuItems = useMemo(
@ -369,7 +369,7 @@ const DashboardNavigation = () => {
<> <>
{isElectron ? ( {isElectron ? (
<Flex <Flex
className={`ant-menu-horizontal ant-menu-light${!notificationCenterVisible && !isFullscreen ? ' electron-navigation-wrapper electrobun-webkit-app-region-drag' : ''}`} className={`ant-menu-horizontal electron-navigation-wrapper ant-menu-light${!notificationCenterVisible && !isFullScreen ? ' electrobun-webkit-app-region-drag' : ''}`}
style={{ lineHeight: '40px', padding: '0 2px 0 2px' }} style={{ lineHeight: '40px', padding: '0 2px 0 2px' }}
> >
{navigationContents} {navigationContents}