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

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

@@ -19,7 +19,7 @@ class _GameHistoryViewState extends State<GameHistoryView> {
late final AppDatabase db; late final AppDatabase db;
late final List<Game> skeletonData = List.filled( late final List<Game> skeletonData = List.filled(
gelbeinhalb marked this conversation as resolved Outdated
flixcoo commented 2025-11-24 11:41:52 +00:00
Outdated
Review
Copy Link

Ich würde die Skeleton-Daten so anpassen, dass du nicht mehr machst als auf den Screen passen. Also bei mir sind das glaub ich so 5

Ich würde die Skeleton-Daten so anpassen, dass du nicht mehr machst als auf den Screen passen. Also bei mir sind das glaub ich so 5
gelbeinhalb commented 2025-11-24 12:36:32 +00:00
Outdated
Review
Copy Link

ja save :) hab 10 eingestellt zum testen und dann vergessen zu ändern

ja save :) hab 10 eingestellt zum testen und dann vergessen zu ändern
2, 10,
Game( Game(
name: 'Skeleton Game', name: 'Skeleton Game',
group: Group( group: Group(
@@ -30,6 +30,9 @@ class _GameHistoryViewState extends State<GameHistoryView> {
], ],
), ),
winner: Player(name: 'Skeleton Player 1'), winner: Player(name: 'Skeleton Player 1'),
players: [
Player(name: 'Skeleton Player 3')
],
), ),
); );
@@ -37,10 +40,11 @@ class _GameHistoryViewState extends State<GameHistoryView> {
void initState() { void initState() {
super.initState(); super.initState();
db = Provider.of<AppDatabase>(context, listen: false); db = Provider.of<AppDatabase>(context, listen: false);
_gameListFuture = Future.delayed( _gameListFuture = db.gameDao.getAllGames();
const Duration(milliseconds: 250),
() => db.gameDao.getAllGames(), Future.wait([_gameListFuture]).then((result) async {
); await Future.delayed(const Duration(milliseconds: 250));
});
} }
@override @override
@@ -69,7 +73,6 @@ class _GameHistoryViewState extends State<GameHistoryView> {
? skeletonData ? skeletonData
: (snapshot.data ?? []) : (snapshot.data ?? [])
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
..sort((a, b) => b.createdAt.compareTo(a.createdAt))) ..sort((a, b) => b.createdAt.compareTo(a.createdAt)))
.take(2)
.toList(); .toList();
return Skeletonizer( return Skeletonizer(