Revert "Threads-style nav: Home, Search, +, Activity (heart+badge), Profile as bottom nav tabs"

This reverts commit 7f81923b3e.
This commit is contained in:
Patrick Britton 2026-02-06 15:58:58 -06:00
parent 7f81923b3e
commit 8623596903
3 changed files with 180 additions and 94 deletions

View file

@ -21,7 +21,6 @@ import '../screens/discover/discover_screen.dart';
import '../screens/secure_chat/secure_chat_full_screen.dart'; import '../screens/secure_chat/secure_chat_full_screen.dart';
import '../screens/secure_chat/secure_chat_loader_screen.dart'; import '../screens/secure_chat/secure_chat_loader_screen.dart';
import '../screens/post/threaded_conversation_screen.dart'; import '../screens/post/threaded_conversation_screen.dart';
import '../screens/notifications/notifications_screen.dart';
/// App routing config (GoRouter). /// App routing config (GoRouter).
class AppRoutes { class AppRoutes {
@ -89,7 +88,6 @@ class AppRoutes {
authenticatedChild: HomeShell(navigationShell: navigationShell), authenticatedChild: HomeShell(navigationShell: navigationShell),
), ),
branches: [ branches: [
// Tab 0: Home
StatefulShellBranch( StatefulShellBranch(
routes: [ routes: [
GoRoute( GoRoute(
@ -98,25 +96,25 @@ class AppRoutes {
), ),
], ],
), ),
// Tab 1: Search / Discover
StatefulShellBranch( StatefulShellBranch(
routes: [ routes: [
GoRoute( GoRoute(
path: '/discover', path: quips,
builder: (_, __) => const DiscoverScreen(), builder: (_, state) => QuipsFeedScreen(
initialPostId: state.uri.queryParameters['postId'],
), ),
),
], ],
), ),
// Tab 2: Activity / Notifications
StatefulShellBranch( StatefulShellBranch(
routes: [ routes: [
GoRoute( GoRoute(
path: '/activity', path: '/beacon',
builder: (_, __) => const NotificationsScreen(), builder: (_, __) => const BeaconScreen(),
), ),
], ],
), ),
// Tab 3: Profile
StatefulShellBranch( StatefulShellBranch(
routes: [ routes: [
GoRoute( GoRoute(

View file

@ -206,13 +206,18 @@ class _HomeShellState extends ConsumerState<HomeShell> with WidgetsBindingObserv
label: 'Home', label: 'Home',
), ),
_buildNavBarItem( _buildNavBarItem(
icon: Icons.search_outlined, icon: Icons.play_circle_outline,
activeIcon: Icons.search, activeIcon: Icons.play_circle,
index: 1, index: 1,
label: 'Search', label: 'Quips',
), ),
const SizedBox(width: 48), const SizedBox(width: 48),
_buildActivityNavItem(index: 2), _buildNavBarItem(
icon: Icons.sensors_outlined,
activeIcon: Icons.sensors,
index: 2,
label: 'Beacon',
),
_buildNavBarItem( _buildNavBarItem(
icon: Icons.person_outline, icon: Icons.person_outline,
activeIcon: Icons.person, activeIcon: Icons.person,
@ -248,6 +253,17 @@ class _HomeShellState extends ConsumerState<HomeShell> with WidgetsBindingObserv
), ),
), ),
actions: [ actions: [
IconButton(
icon: Icon(Icons.search, color: AppTheme.navyBlue),
tooltip: 'Discover',
onPressed: () {
Navigator.of(context).push(
MaterialPageRoute(
builder: (_) => const DiscoverScreen(),
),
);
},
),
IconButton( IconButton(
icon: Consumer( icon: Consumer(
builder: (context, ref, child) { builder: (context, ref, child) {
@ -270,6 +286,27 @@ class _HomeShellState extends ConsumerState<HomeShell> with WidgetsBindingObserv
); );
}, },
), ),
IconButton(
icon: Consumer(
builder: (context, ref, child) {
final badge = ref.watch(currentBadgeProvider);
return Badge(
label: Text(badge.notificationCount.toString()),
isLabelVisible: badge.notificationCount > 0,
backgroundColor: Colors.redAccent,
child: Icon(Icons.notifications_none, color: AppTheme.navyBlue),
);
},
),
tooltip: 'Notifications',
onPressed: () {
Navigator.of(context).push(
MaterialPageRoute(
builder: (_) => const NotificationsScreen(),
),
);
},
),
const SizedBox(width: 4), const SizedBox(width: 4),
], ],
); );
@ -300,37 +337,6 @@ class _HomeShellState extends ConsumerState<HomeShell> with WidgetsBindingObserv
), ),
); );
} }
Widget _buildActivityNavItem({required int index}) {
final isActive = widget.navigationShell.currentIndex == index;
return Expanded(
child: InkWell(
onTap: () => widget.navigationShell.goBranch(
index,
initialLocation: index == widget.navigationShell.currentIndex,
),
child: Container(
height: double.infinity,
alignment: Alignment.center,
child: Consumer(
builder: (context, ref, child) {
final badge = ref.watch(currentBadgeProvider);
return Badge(
label: Text(badge.notificationCount.toString()),
isLabelVisible: badge.notificationCount > 0,
backgroundColor: Colors.redAccent,
child: Icon(
isActive ? Icons.favorite : Icons.favorite_border,
color: isActive ? AppTheme.navyBlue : Colors.grey,
size: 26,
),
);
},
),
),
),
);
}
} }
class _VerticalBorderProgressPainter extends CustomPainter { class _VerticalBorderProgressPainter extends CustomPainter {

View file

@ -6,9 +6,13 @@ import 'package:timeago/timeago.dart' as timeago;
import '../../models/notification.dart'; import '../../models/notification.dart';
import '../../providers/api_provider.dart'; import '../../providers/api_provider.dart';
import '../../theme/app_theme.dart'; import '../../theme/app_theme.dart';
import '../../widgets/app_scaffold.dart';
import '../../widgets/media/signed_media_image.dart'; import '../../widgets/media/signed_media_image.dart';
import '../profile/viewable_profile_screen.dart'; import '../profile/viewable_profile_screen.dart';
import '../post/post_detail_screen.dart'; import '../post/post_detail_screen.dart';
import '../search/search_screen.dart';
import '../discover/discover_screen.dart';
import '../secure_chat/secure_chat_full_screen.dart';
import 'package:go_router/go_router.dart'; import 'package:go_router/go_router.dart';
import '../../services/notification_service.dart'; import '../../services/notification_service.dart';
@ -348,29 +352,35 @@ class _NotificationsScreenState extends ConsumerState<NotificationsScreen> {
} }
} }
void _navigateHome() {
Navigator.of(context).pop();
}
void _navigateSearch() {
Navigator.of(context).pushReplacement(
MaterialPageRoute(builder: (_) => const DiscoverScreen()),
);
}
void _navigateChat() {
Navigator.of(context).pushReplacement(
MaterialPageRoute(
builder: (_) => const SecureChatFullScreen(),
fullscreenDialog: true,
),
);
}
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
final canArchiveAll = _activeTabIndex == 0 && _notifications.isNotEmpty; final canArchiveAll = _activeTabIndex == 0 && _notifications.isNotEmpty;
return DefaultTabController( return DefaultTabController(
length: 2, length: 2,
child: Scaffold( child: AppScaffold(
backgroundColor: AppTheme.scaffoldBg, title: 'Activity',
body: SafeArea( leading: const SizedBox.shrink(),
child: Column( actions: [
children: [
// Header
Padding(
padding: const EdgeInsets.symmetric(horizontal: 16, vertical: 8),
child: Row(
children: [
Text(
'Activity',
style: AppTheme.textTheme.headlineSmall?.copyWith(
fontWeight: FontWeight.bold,
),
),
const Spacer(),
if (canArchiveAll) if (canArchiveAll)
TextButton( TextButton(
onPressed: _archiveAllNotifications, onPressed: _archiveAllNotifications,
@ -383,10 +393,7 @@ class _NotificationsScreenState extends ConsumerState<NotificationsScreen> {
), ),
), ),
], ],
), bottom: TabBar(
),
// Tab bar
TabBar(
onTap: (index) { onTap: (index) {
if (index != _activeTabIndex) { if (index != _activeTabIndex) {
setState(() { setState(() {
@ -403,9 +410,8 @@ class _NotificationsScreenState extends ConsumerState<NotificationsScreen> {
Tab(text: 'Archived'), Tab(text: 'Archived'),
], ],
), ),
// Content bottomNavigationBar: _buildBottomNav(),
Expanded( body: _error != null
child: _error != null
? _ErrorState( ? _ErrorState(
message: _error!, message: _error!,
onRetry: () => _loadNotifications(refresh: true), onRetry: () => _loadNotifications(refresh: true),
@ -480,12 +486,88 @@ class _NotificationsScreenState extends ConsumerState<NotificationsScreen> {
), ),
), ),
), ),
);
}
Widget _buildBottomNav() {
return Container(
decoration: BoxDecoration(
color: AppTheme.scaffoldBg,
border: Border(
top: BorderSide(
color: AppTheme.egyptianBlue.withOpacity(0.1),
width: 0.5,
),
),
),
child: SafeArea(
top: false,
child: Padding(
padding: const EdgeInsets.symmetric(vertical: 8),
child: Row(
mainAxisAlignment: MainAxisAlignment.spaceAround,
children: [
_buildNavItem(
icon: Icons.home_outlined,
label: 'Home',
onTap: _navigateHome,
),
_buildNavItem(
icon: Icons.search,
label: 'Discover',
onTap: _navigateSearch,
),
_buildNavItem(
icon: Icons.notifications,
label: 'Activity',
isActive: true,
onTap: () {},
),
_buildNavItem(
icon: Icons.chat_bubble_outline,
label: 'Chat',
onTap: _navigateChat,
),
], ],
), ),
), ),
), ),
); );
} }
Widget _buildNavItem({
required IconData icon,
required String label,
required VoidCallback onTap,
bool isActive = false,
}) {
return InkWell(
onTap: onTap,
borderRadius: BorderRadius.circular(12),
child: Padding(
padding: const EdgeInsets.symmetric(horizontal: 16, vertical: 4),
child: Column(
mainAxisSize: MainAxisSize.min,
children: [
Icon(
icon,
color: isActive ? AppTheme.navyBlue : Colors.grey,
size: 26,
),
const SizedBox(height: 2),
Text(
label,
style: TextStyle(
fontSize: 10,
color: isActive ? AppTheme.navyBlue : Colors.grey,
fontWeight: isActive ? FontWeight.w600 : FontWeight.normal,
),
),
],
),
),
);
}
} }
/// Individual notification item widget /// Individual notification item widget