From 9a0979810077863e81578b57f3c255eff58e98eb Mon Sep 17 00:00:00 2001
From: Mistral Vibe <vibe@mistral.ai>
Date: Tue, 7 Apr 2026 13:35:09 +0000
Subject: [PATCH] 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.
---
 web/src/components/BottomNavBar.tsx | 17 ++---------------
 1 file changed, 2 insertions(+), 15 deletions(-)

diff --git a/web/src/components/BottomNavBar.tsx b/web/src/components/BottomNavBar.tsx
index 75985ba..fb76adf 100644
--- a/web/src/components/BottomNavBar.tsx
+++ b/web/src/components/BottomNavBar.tsx
@@ -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"