liquid-development/game-tracker
3
0
Fork 0
Code Issues 14 Pull Requests 7 Actions Releases 1 Activity

Navbar Anpassen #9

Merged
gelbeinhalb merged 11 commits from enhancement/8-tabbar-angleichen into development 2025-11-09 19:59:56 +00:00
Conversation 2 Commits 11 Files Changed 2 +26 -20
1 changed files with 26 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 17d304eb5d - Show all commits

8
lib/presentation/views/main_menu/custom_navigation_bar.dart
View File

@@ -82,8 +82,12 @@ class _CustomNavigationBarState extends State<CustomNavigationBar>
Widget _buildNavItem(IconData icon, String label, int index) { Widget _buildNavItem(IconData icon, String label, int index) {
flixcoo marked this conversation as resolved Outdated
flixcoo commented 2025-11-08 16:38:17 +00:00
Outdated
Review
Copy Link

Ich würde das Widget entweder Auslagern (als nav_item.dart im widgets Ordner) oder das Expanded Widget direkt in die Row setzen (nicht so geil). Wenn du das Auslagerst sparst du dir ganz viel Code hier und übergibst die Parameter einfach direkt in das Widget.

Ich würde das Widget entweder Auslagern (als `nav_item.dart` im `widgets` Ordner) oder das `Expanded` Widget direkt in die Row setzen (nicht so geil). Wenn du das Auslagerst sparst du dir ganz viel Code hier und übergibst die Parameter einfach direkt in das Widget.
gelbeinhalb commented 2025-11-08 16:43:37 +00:00
Outdated
Review
Copy Link

Ja stimmt.
Ich lager das dann aus.

Ja stimmt. Ich lager das dann aus.
final isSelected = currentIndex == index; final isSelected = currentIndex == index;
return GestureDetector( return Expanded( // makes each nav item occupy equal width = large horizontal hitbox
gelbeinhalb marked this conversation as resolved Outdated
flixcoo commented 2025-11-08 16:39:04 +00:00
Outdated
Review
Copy Link

Chat GPT kommentare?

Chat GPT kommentare?
child: GestureDetector(
onTap: () => onTabTapped(index), onTap: () => onTabTapped(index),
behavior: HitTestBehavior.opaque, // ensures the entire area is tappable
child: Padding(
padding: const EdgeInsets.symmetric(vertical: 5.0), // adds comfortable tap height
child: Column( child: Column(
mainAxisSize: MainAxisSize.min, mainAxisSize: MainAxisSize.min,
mainAxisAlignment: MainAxisAlignment.center, mainAxisAlignment: MainAxisAlignment.center,
@@ -103,6 +107,8 @@ class _CustomNavigationBarState extends State<CustomNavigationBar>
), ),
], ],
), ),
),
),
); );
} }