Merge remote-tracking branch 'origin/development' into feature/2-gamehistoryview-anpassen

Merge pull request 'Gelbes Aufblinken der Recent Games auf der Homepage behoben & Anpassung des Recent-Games-Containers vorgenommen.' (#64 ) from bug/63-übergang-auf-home-seite-blitz-kurz-gelb-auf into development
Reviewed-on: #64 Reviewed-by: Felix Kirchner <felix.kirchner.fk@gmail.com>
2025-11-24 11:34:59 +01:00 · 2025-11-23 21:51:04 +00:00 · 2025-11-23 21:48:48 +00:00 · 2025-11-23 21:41:08 +00:00 · 2025-11-23 22:14:13 +01:00 · 2025-11-23 21:16:13 +01:00
2 changed files with 8 additions and 16 deletions
--- a/lib/presentation/views/main_menu/create_group_view.dart
+++ b/lib/presentation/views/main_menu/create_group_view.dart
@@ -209,12 +209,13 @@ class _CreateGroupViewState extends State<CreateGroupView> {
                                ),
                              );
                            }
-                            if (snapshot.connectionState ==
-                                    ConnectionState.done &&
-                                (!snapshot.hasData ||
-                                    snapshot.data!.isEmpty ||
-                                    (selectedPlayers.isEmpty &&
-                                        allPlayers.isEmpty))) {
+                            bool doneLoading =
+                                snapshot.connectionState ==
+                                ConnectionState.done;
+                            bool snapshotDataEmpty =
+                                !snapshot.hasData || snapshot.data!.isEmpty;
+                            if (doneLoading &&
+                                (snapshotDataEmpty && allPlayers.isEmpty)) {
                              return const Center(
                                child: TopCenteredMessage(
                                  icon: Icons.info,
--- a/lib/presentation/views/main_menu/home_view.dart
+++ b/lib/presentation/views/main_menu/home_view.dart
@@ -151,15 +151,6 @@ class _HomeViewState extends State<HomeView> {
                                  ),
                                );
                              }
-                              if (snapshot.connectionState ==
-                                      ConnectionState.done &&
-                                  (!snapshot.hasData ||
-                                      snapshot.data!.isEmpty)) {
-                                return const Center(
-                                  heightFactor: 4,
-                                  child: Text('No recent games available.'),
-                                );
-                              }
                              final List<Game> games =
                                  (isLoading
                                            ? skeletonData
@@ -214,7 +205,7 @@ class _HomeViewState extends State<HomeView> {
                                );
                              } else {
                                return const Center(
-                                  heightFactor: 4,
+                                  heightFactor: 12,
                                  child: Text('No recent games available.'),
                                );
                              }
Author	SHA1	Message	Date
Yannick	44279bc148	Merge remote-tracking branch 'origin/development' into feature/2-gamehistoryview-anpassen Some checks failed Pull Request Pipeline / test (pull_request) Successful in 2m15s Details Pull Request Pipeline / lint (pull_request) Failing after 2m15s Details	2025-11-24 11:34:59 +01:00
Felix Kirchner	a3fa499662	Merge pull request 'Gelbes Aufblinken der Recent Games auf der Homepage behoben & Anpassung des Recent-Games-Containers vorgenommen.' (#64 ) from bug/63-übergang-auf-home-seite-blitz-kurz-gelb-auf into development Reviewed-on: #64 Reviewed-by: Felix Kirchner <felix.kirchner.fk@gmail.com>	2025-11-23 21:51:04 +00:00
mathiskir	f25737cdc5	Merge branch 'development' into bug/63-übergang-auf-home-seite-blitz-kurz-gelb-auf All checks were successful Pull Request Pipeline / test (pull_request) Successful in 2m9s Details Pull Request Pipeline / lint (pull_request) Successful in 2m9s Details	2025-11-23 21:48:48 +00:00
Felix Kirchner	0b500b5248	Merge pull request 'Bugfix bei Spieleranzeige und Vereinfachung der Keine-Player-Logik' (#66 ) from bug/60-CreateGroupView-erstellter-Spieler-erscheint-nicht-in-alle-Spieler into development Reviewed-on: #66 Reviewed-by: Felix Kirchner <felix.kirchner.fk@gmail.com>	2025-11-23 21:41:08 +00:00
mathiskirchner	974d6e9b56	refactor empty state logic in CreateGroupView All checks were successful Pull Request Pipeline / test (pull_request) Successful in 2m22s Details Pull Request Pipeline / lint (pull_request) Successful in 2m24s Details The diff introduces boolean variables `doneLoading` and `snapshotDataEmpty` to simplify the conditional check for displaying the empty state message. It specifically fixes the logic to correctly show the "No players found" message when both the snapshot and the local `allPlayers` list are empty, removing the dependency on `selectedPlayers.isEmpty`.	2025-11-23 22:14:13 +01:00
mathiskirchner	694cac7f26	Fix bug where the skeleton was edited while it was visible and All checks were successful Pull Request Pipeline / test (pull_request) Successful in 2m4s Details Pull Request Pipeline / lint (pull_request) Successful in 2m6s Details match the “no games available” container size to the size used when games are available.	2025-11-23 21:16:13 +01:00