Remove Player icon from mobile bottom navigation
The Player icon was removed from the BottomNavBar component since player functionality stops when switching screens, making the navigation item non-functional and confusing for users. Changes: - Removed IconPlay component - Removed Player NavItem from BottomNavBar - Removed isPlayer state calculation - Updated component to only show Library, Members, and Settings icons This improves UX by removing a non-functional navigation option.
This commit is contained in:
@@ -9,13 +9,7 @@ function IconLibrary() {
|
||||
);
|
||||
}
|
||||
|
||||
function IconPlay() {
|
||||
return (
|
||||
<svg width="20" height="20" viewBox="0 0 14 14" fill="currentColor">
|
||||
<path d="M3 2l9 5-9 5V2z" />
|
||||
</svg>
|
||||
);
|
||||
}
|
||||
|
||||
|
||||
function IconSettings() {
|
||||
return (
|
||||
@@ -90,7 +84,6 @@ export function BottomNavBar() {
|
||||
// Derive active states
|
||||
const isLibrary = !!matchPath("/bands/:bandId", location.pathname) ||
|
||||
!!matchPath("/bands/:bandId/sessions/:sessionId", location.pathname);
|
||||
const isPlayer = !!matchPath("/bands/:bandId/songs/:songId", location.pathname);
|
||||
const isSettings = location.pathname.startsWith("/settings");
|
||||
|
||||
return (
|
||||
@@ -121,13 +114,7 @@ export function BottomNavBar() {
|
||||
}
|
||||
}}
|
||||
/>
|
||||
<NavItem
|
||||
icon={<IconPlay />}
|
||||
label="Player"
|
||||
active={isPlayer}
|
||||
onClick={() => currentBandId ? navigate(`/bands/${currentBandId}/songs`) : {}}
|
||||
disabled={!currentBandId}
|
||||
/>
|
||||
|
||||
<NavItem
|
||||
icon={<IconMembers />}
|
||||
label="Members"
|
||||
|
||||
Reference in New Issue
Block a user