From 570376259e62aecf4ca4553d213d7722a24ed42f Mon Sep 17 00:00:00 2001
From: PartyDonut <PartyDonut@users.noreply.github.com>
Date: Sun, 6 Oct 2024 19:35:57 +0200
Subject: [PATCH] [Bugfix] Fixed season details screen UI bugs

---
 .../details_screens/season_detail_screen.dart | 77 +++++++++++--------
 1 file changed, 44 insertions(+), 33 deletions(-)

diff --git a/lib/screens/details_screens/season_detail_screen.dart b/lib/screens/details_screens/season_detail_screen.dart
index ec2874f..b6629f7 100644
--- a/lib/screens/details_screens/season_detail_screen.dart
+++ b/lib/screens/details_screens/season_detail_screen.dart
@@ -19,6 +19,7 @@ import 'package:fladder/util/item_base_model/item_base_model_extensions.dart';
 import 'package:fladder/util/list_padding.dart';
 import 'package:fladder/util/localization_helper.dart';
 import 'package:fladder/util/string_extensions.dart';
+import 'package:fladder/util/theme_extensions.dart';
 import 'package:fladder/util/widget_extensions.dart';
 import 'package:fladder/widgets/shared/selectable_icon_button.dart';
 
@@ -55,12 +56,21 @@ class _SeasonDetailScreenState extends ConsumerState<SeasonDetailScreen> {
             ? Column(
                 crossAxisAlignment: CrossAxisAlignment.stretch,
                 children: [
-                  SizedBox(height: MediaQuery.of(context).size.height * 0.35),
+                  SizedBox(height: MediaQuery.of(context).size.height * 0.25),
                   Wrap(
                     alignment: WrapAlignment.spaceAround,
                     runAlignment: WrapAlignment.center,
                     crossAxisAlignment: WrapCrossAlignment.center,
                     children: [
+                      ConstrainedBox(
+                        constraints: const BoxConstraints(maxWidth: 300),
+                        child: AspectRatio(
+                          aspectRatio: 0.67,
+                          child: Card(
+                            child: FladderImage(image: details.getPosters?.primary),
+                          ),
+                        ),
+                      ),
                       ConstrainedBox(
                         constraints: const BoxConstraints(
                           maxWidth: 600,
@@ -89,9 +99,6 @@ class _SeasonDetailScreenState extends ConsumerState<SeasonDetailScreen> {
                           ],
                         ),
                       ),
-                      ConstrainedBox(
-                          constraints: const BoxConstraints(maxWidth: 300),
-                          child: Card(child: FladderImage(image: details.getPosters?.primary))),
                     ],
                   ).padding(padding),
                   Row(
@@ -123,36 +130,40 @@ class _SeasonDetailScreenState extends ConsumerState<SeasonDetailScreen> {
                       ),
                       Row(
                         children: [
-                          Card(
-                            shape: RoundedRectangleBorder(borderRadius: BorderRadius.circular(200)),
-                            child: SegmentedButton(
-                              style: const ButtonStyle(
-                                elevation: WidgetStatePropertyAll(5),
-                                side: WidgetStatePropertyAll(BorderSide.none),
-                              ),
-                              showSelectedIcon: true,
-                              segments: EpisodeDetailsViewType.values
-                                  .map(
-                                    (e) => ButtonSegment(
-                                      value: e,
-                                      icon: Icon(e.icon),
-                                      label: SizedBox(
-                                          height: 50,
-                                          child: Center(
-                                            child: Text(
-                                              e.name.capitalize(),
-                                            ),
-                                          )),
-                                    ),
-                                  )
-                                  .toList(),
-                              selected: viewOptions,
-                              onSelectionChanged: (newOptions) {
-                                setState(() {
-                                  viewOptions = newOptions;
-                                });
-                              },
+                          SegmentedButton(
+                            style: ButtonStyle(
+                              backgroundColor: WidgetStateProperty.resolveWith((state) {
+                                if (state.contains(WidgetState.selected)) {
+                                  return context.colors.primaryContainer;
+                                }
+                                return context.colors.surfaceContainer;
+                              }),
+                              padding: const WidgetStatePropertyAll(EdgeInsets.symmetric(vertical: 8, horizontal: 16)),
+                              elevation: const WidgetStatePropertyAll(5),
+                              side: const WidgetStatePropertyAll(BorderSide.none),
                             ),
+                            showSelectedIcon: true,
+                            segments: EpisodeDetailsViewType.values
+                                .map(
+                                  (e) => ButtonSegment(
+                                    value: e,
+                                    icon: Icon(e.icon),
+                                    label: SizedBox(
+                                        height: 40,
+                                        child: Center(
+                                          child: Text(
+                                            e.name.capitalize(),
+                                          ),
+                                        )),
+                                  ),
+                                )
+                                .toList(),
+                            selected: viewOptions,
+                            onSelectionChanged: (newOptions) {
+                              setState(() {
+                                viewOptions = newOptions;
+                              });
+                            },
                           ),
                         ],
                       ),