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

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

@@ -4,6 +4,7 @@ import 'package:game_tracker/data/dto/game.dart';
import 'package:game_tracker/data/dto/group.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/top_centered_message.dart';
import 'package:provider/provider.dart';
import 'package:skeletonizer/skeletonizer.dart';
@@ -61,17 +62,21 @@ class _GameHistoryViewState extends State<GameHistoryView> {
builder: (BuildContext context, AsyncSnapshot<List<Game>> snapshot) {
gelbeinhalb marked this conversation as resolved Outdated
sneeex commented 2025-11-24 11:58:53 +00:00
Outdated
Review
Copy Link

Errormessage anders, handelt sich ja nicht um recent games

Errormessage anders, handelt sich ja nicht um recent games
gelbeinhalb commented 2025-11-24 12:43:36 +00:00
Outdated
Review
Copy Link

jo

jo
sneeex commented 2025-11-24 15:27:33 +00:00
Outdated
Review
Copy Link

und besser standartisierte top_centered_message wie in GroupsView benutzen.
Außerdem sieht das Alignement komisch aus:
grafik.png

und besser standartisierte `top_centered_message` wie in GroupsView benutzen. Außerdem sieht das Alignement komisch aus: ![grafik.png](/attachments/51e0ad5c-7df8-43e1-8401-dd459be4abbb)
grafik.png
52 KiB
gelbeinhalb commented 2025-11-24 15:28:04 +00:00
Outdated
Review
Copy Link

hab das 1zu1 von felix kopiert

hab das 1zu1 von felix kopiert
flixcoo commented 2025-11-24 16:51:20 +00:00
Outdated
Review
Copy Link

Ja das war ein schwerer Fehler, würde nichtmal von mir selbst kopieren

Ja das war ein schwerer Fehler, würde nichtmal von mir selbst kopieren
if (snapshot.hasError) {
return const Center(
heightFactor: 4,
child: Text(
'Error while loading games.',
child: TopCenteredMessage(
icon: Icons.report,
title: 'Error',
message: 'Game data could not be loaded',
),
);
gelbeinhalb marked this conversation as resolved Outdated
sneeex commented 2025-11-24 11:58:59 +00:00
Outdated
Review
Copy Link

Errormessage anders, handelt sich ja nicht um recent games

Errormessage anders, handelt sich ja nicht um recent games
gelbeinhalb commented 2025-11-24 12:43:47 +00:00
Outdated
Review
Copy Link

jo

jo
sneeex commented 2025-11-24 15:27:43 +00:00
Outdated
Review
Copy Link

und besser standartisierte top_centered_message wie in GroupsView benutzen.
Außerdem sieht das Alignement komisch aus:
grafik.png

und besser standartisierte `top_centered_message` wie in GroupsView benutzen. Außerdem sieht das Alignement komisch aus: ![grafik.png](/attachments/51e0ad5c-7df8-43e1-8401-dd459be4abbb)
}
if (snapshot.connectionState == ConnectionState.done &&
(!snapshot.hasData || snapshot.data!.isEmpty)) {
return const Center(
gelbeinhalb marked this conversation as resolved Outdated
flixcoo commented 2025-11-24 11:39:55 +00:00
Outdated
Review
Copy Link

Die Skeleton-Zeit ist viel kürzer als bei allen anderen Screens. Ich glaube das liegt an dieser Zeile, weil du hier die isLoading Variable setzt und nicht nach dem oben angegebenen 250ms delay.

Die Skeleton-Zeit ist viel kürzer als bei allen anderen Screens. Ich glaube das liegt an dieser Zeile, weil du hier die `isLoading` Variable setzt und nicht nach dem oben angegebenen 250ms delay.
gelbeinhalb commented 2025-11-24 12:37:02 +00:00
Outdated
Review
Copy Link

okay danke :) guck ich mir an

okay danke :) guck ich mir an
heightFactor: 4,
child: Text('No games available.'),
child: TopCenteredMessage(
icon: Icons.report,
title: 'Error',
message: 'No Games Available',
),
);
}