TT-7036 Update ProjectMenu to conditionally render menu items based o… #188
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the ProjectMenu component to conditionally hide certain menu items on mobile devices (screens smaller than the 'sm' breakpoint). The change uses MUI's useMediaQuery and useTheme hooks to determine when the viewport is mobile-sized and applies the !isMobileWidth condition to most admin-related menu items.
Changes:
- Added useMediaQuery and useTheme imports from @mui/material
- Added mobile width detection using theme breakpoints
- Conditionally rendered settings, copy, integrations, category edit, import, and delete menu items based on mobile width
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| {!justFilter && | ||
| !isMobileWidth && | ||
| pathname && | ||
| projType.toLowerCase() === 'scripture' && | ||
| pathname.indexOf(ArtifactTypeSlug.Retell) === -1 && |
There was a problem hiding this comment.
The offline availability menu item (lines 148-159) is missing the mobile width check that was added to other menu items. This creates an inconsistency in which menu items are shown on mobile devices. Consider whether this option should also be hidden on mobile for consistency with the other admin-related features.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…n mobile width