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

CreateGroupView erstellt #28

Merged
flixcoo merged 37 commits from feature/5-creategroupview-erstellen into development 2025-11-19 17:32:44 +00:00
Conversation 13 Commits 37 Files Changed 16 +2 -2
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 248d652e06 - Show all commits

4
lib/presentation/widgets/tiles/text_icon_list_tile.dart
View File

@@ -3,13 +3,13 @@ import 'package:game_tracker/core/custom_theme.dart';
class TextIconListTile extends StatelessWidget {
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:37:29 +00:00
Outdated
Review
Copy Link

Klassenname stimmt nicht mit Dateiname überein

Klassenname stimmt nicht mit Dateiname überein
final String text;
final VoidCallback onPressed;
final VoidCallback? onPressed;
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:39:48 +00:00
Outdated
Review
Copy Link

Falls das Icon bisher nur ein Icon ist, ggf. nur das Icon deaktivierbar machen anstatt es komplett selbst zu setzen

Falls das Icon bisher nur ein Icon ist, ggf. nur das Icon deaktivierbar machen anstatt es komplett selbst zu setzen
final bool iconEnabled;
const TextIconListTile({
sneeex marked this conversation as resolved
flixcoo commented 2025-11-19 00:01:45 +00:00
Review
Copy Link

grafik.png

Alignment irgendwie komisch, Abstand vom Button zum Rand viel größer als vom Namen zum rand

![grafik.png](/attachments/a1c43803-f0d1-4a01-9ab5-e2e43c651ee5) Alignment irgendwie komisch, Abstand vom Button zum Rand viel größer als vom Namen zum rand
grafik.png
8.6 KiB
super.key,
required this.text,
required this.onPressed,
this.onPressed,
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-19 00:00:58 +00:00
Outdated
Review
Copy Link

onPressed sollte nich required sein wenn das icon nicht immer zu sehen ist

onPressed sollte nich required sein wenn das icon nicht immer zu sehen ist
this.iconEnabled = true,
});