liquid-development/game-tracker
3
0
Fork 0
Code Issues 14 Pull Requests 6 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 +4 -4
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 1698f61c2b - Show all commits

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

@@ -52,7 +52,7 @@ class _CustomNavigationBarState extends State<CustomNavigationBar>
),
backgroundColor: CustomTheme.backgroundColor,
body: tabs[currentIndex],
extendBody: true, // Enables floating effect
extendBody: true,
flixcoo marked this conversation as resolved Outdated
flixcoo commented 2025-11-08 16:35:06 +00:00
Outdated
Review
Copy Link

Chat GPT Kommentar entfernen

Chat GPT Kommentar entfernen
gelbeinhalb commented 2025-11-08 16:42:32 +00:00
Outdated
Review
Copy Link

Hab ich extra reingemacht weil ich das so unlogisch fand. Also der Code erklärt sich nicht selber finde ich

Hab ich extra reingemacht weil ich das so unlogisch fand. Also der Code erklärt sich nicht selber finde ich
bottomNavigationBar: Padding(
padding: const EdgeInsets.only(left: 12.0, right: 12.0, bottom: 8.0),
child: Material(
@@ -82,12 +82,12 @@ class _CustomNavigationBarState extends State<CustomNavigationBar>
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;
return Expanded( // makes each nav item occupy equal width = large horizontal hitbox
return Expanded(
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),
behavior: HitTestBehavior.opaque, // ensures the entire area is tappable
behavior: HitTestBehavior.opaque,
child: Padding(
padding: const EdgeInsets.symmetric(vertical: 5.0), // adds comfortable tap height
padding: const EdgeInsets.symmetric(vertical: 5.0),
child: Column(
mainAxisSize: MainAxisSize.min,
mainAxisAlignment: MainAxisAlignment.center,