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

GameHistoryView anpassen #20

Merged
flixcoo merged 27 commits from feature/2-gamehistoryview-anpassen into development 2025-11-30 15:59:25 +00:00
Conversation 18 Commits 27 Files Changed 4 +2 -15
1 changed files with 2 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit f713bd6fb7 - Show all commits

17
lib/presentation/views/main_menu/game_history_view.dart
View File

@@ -5,8 +5,8 @@ import 'package:game_tracker/data/dto/group.dart';
import 'package:game_tracker/data/dto/player.dart'; import 'package:game_tracker/data/dto/player.dart';
import 'package:game_tracker/presentation/widgets/tiles/game_history_tile.dart'; import 'package:game_tracker/presentation/widgets/tiles/game_history_tile.dart';
import 'package:game_tracker/presentation/widgets/top_centered_message.dart'; import 'package:game_tracker/presentation/widgets/top_centered_message.dart';
import 'package:game_tracker/presentation/widgets/app_skeleton.dart'; // Add this import
import 'package:provider/provider.dart'; import 'package:provider/provider.dart';
import 'package:skeletonizer/skeletonizer.dart';
class GameHistoryView extends StatefulWidget { class GameHistoryView extends StatefulWidget {
gelbeinhalb marked this conversation as resolved Outdated
flixcoo commented 2025-11-27 22:25:25 +00:00
Outdated
Review
Copy Link

Umbennenung zu GameView stand mal im Raum oder?

Umbennenung zu `GameView` stand mal im Raum oder?
gelbeinhalb commented 2025-11-27 22:28:44 +00:00
Outdated
Review
Copy Link

Ist nicht ein GameView das was sich öffnet wenn man auf ein Game klickt?

Ist nicht ein GameView das was sich öffnet wenn man auf ein Game klickt?
flixcoo commented 2025-11-27 22:50:11 +00:00
Outdated
Review
Copy Link

@mathiskir hat die jetzt GameResultView genannt, aber quatsch ihr euch da sonst nochmal ab

@mathiskir hat die jetzt `GameResultView` genannt, aber quatsch ihr euch da sonst nochmal ab
gelbeinhalb commented 2025-11-28 08:57:34 +00:00
Outdated
Review
Copy Link

Ich finde wir sollten beides von Game....View in Match...View umbenennen.

Game = Das Spiel (Brettspiel, Kartenspiel)
Match = Eine Partie des Spiels

@flixcoo @mathiskir

Ich finde wir sollten beides von `Game....View` in `Match...View` umbenennen. Game = Das Spiel (Brettspiel, Kartenspiel) Match = Eine Partie des Spiels @flixcoo @mathiskir
flixcoo commented 2025-11-28 09:09:59 +00:00
Outdated
Review
Copy Link

Ja fand ich glaub ich sinnvoll. Dann müssten wir aber auch die entsprechenden Klassen umbenennen. Ich würde vorschlagen dass gesammelt in einem Ticket zu machen und deins so zu mergen

Ja fand ich glaub ich sinnvoll. Dann müssten wir aber auch die entsprechenden Klassen umbenennen. Ich würde vorschlagen dass gesammelt in einem Ticket zu machen und deins so zu mergen
gelbeinhalb commented 2025-11-28 11:16:22 +00:00
Outdated
Review
Copy Link

Ja würde ich auch sagen, weil dann ja sogar die Datenbank implementation geändert werden muss noch. Und auch unsere test json

Ja würde ich auch sagen, weil dann ja sogar die Datenbank implementation geändert werden muss noch. Und auch unsere test json
sneeex commented 2025-11-28 12:24:35 +00:00
Outdated
Review
Copy Link

ja, lass das hier erstmal rauslassen

ja, lass das hier erstmal rauslassen
const GameHistoryView({super.key}); const GameHistoryView({super.key});
@@ -86,21 +86,8 @@ class _GameHistoryViewState extends State<GameHistoryView> {
..sort((a, b) => b.createdAt.compareTo(a.createdAt))) ..sort((a, b) => b.createdAt.compareTo(a.createdAt)))
.toList(); .toList();
return Skeletonizer( return AppSkeleton(
gelbeinhalb marked this conversation as resolved Outdated
flixcoo commented 2025-11-27 22:23:38 +00:00
Outdated
Review
Copy Link

Bitte das Custom Widget AppSkeleton aus development hier noch Implementieren

Bitte das Custom Widget `AppSkeleton` aus `development` hier noch Implementieren
👍 1
effect: PulseEffect(
from: Colors.grey[800]!,
to: Colors.grey[600]!,
duration: const Duration(milliseconds: 800),
),
enabled: isLoading, enabled: isLoading,
enableSwitchAnimation: true,
switchAnimationConfig: const SwitchAnimationConfig(
duration: Duration(milliseconds: 200),
switchInCurve: Curves.linear,
switchOutCurve: Curves.linear,
transitionBuilder: AnimatedSwitcher.defaultTransitionBuilder,
layoutBuilder: AnimatedSwitcher.defaultLayoutBuilder,
),
child: ListView.builder( child: ListView.builder(
padding: const EdgeInsets.only(bottom: 85), padding: const EdgeInsets.only(bottom: 85),
itemCount: games.length + 1, itemCount: games.length + 1,