Merge 42e954428b into c709de6bf1

chore: release 3.9.0

lib/components/animated_gradient.dart

@@ -1,126 +0,0 @@
-import 'package:shadcn_flutter/shadcn_flutter.dart';
-import 'package:flutter_hooks/flutter_hooks.dart';
-
-class AnimateGradient extends HookWidget {
-  const AnimateGradient({
-    super.key,
-    required this.primaryColors,
-    required this.secondaryColors,
-    this.child,
-    this.primaryBegin,
-    this.primaryEnd,
-    this.secondaryBegin,
-    this.secondaryEnd,
-    AnimationController? controller,
-    this.duration = const Duration(seconds: 4),
-    this.animateAlignments = true,
-    this.reverse = true,
-  })  : assert(primaryColors.length >= 2),
-        assert(primaryColors.length == secondaryColors.length),
-        _controller = controller;
-
-  /// [controller]: pass this to have a fine control over the [Animation]
-  final AnimationController? _controller;
-
-  /// [duration]: Time to switch between [Gradient].
-  /// By default its value is [Duration(seconds:4)]
-  final Duration duration;
-
-  /// [primaryColors]: These will be the starting colors of the [Animation].
-  final List<Color> primaryColors;
-
-  /// [secondaryColors]: These Colors are those in which the [primaryColors] will transition into.
-  final List<Color> secondaryColors;
-
-  /// [primaryBegin]: This is begin [Alignment] for [primaryColors].
-  /// By default its value is [Alignment.topLeft]
-  final Alignment? primaryBegin;
-
-  /// [primaryBegin]: This is end [Alignment] for [primaryColors].
-  /// By default its value is [Alignment.topRight]
-  final Alignment? primaryEnd;
-
-  /// [secondaryBegin]: This is begin [Alignment] for [secondaryColors].
-  /// By default its value is [Alignment.bottomLeft]
-  final Alignment? secondaryBegin;
-
-  /// [secondaryEnd]: This is end [Alignment] for [secondaryColors].
-  /// By default its value is [Alignment.bottomRight]
-  final Alignment? secondaryEnd;
-
-  /// [animateAlignments]: set to false if you don't want to animate the alignments.
-  /// This can provide you way cooler animations
-  final bool animateAlignments;
-
-  /// [reverse]: set it to false if you don't want to reverse the animation.
-  /// using that it will go into one direction only
-  final bool reverse;
-
-  final Widget? child;
-
-  @override
-  Widget build(BuildContext context) {
-    // ignore: no_leading_underscores_for_local_identifiers
-    final __controller = useAnimationController(
-      duration: duration,
-    )..repeat(reverse: reverse);
-
-    final controller = _controller ?? __controller;
-
-    final animation = useMemoized(
-        () => CurvedAnimation(
-              parent: controller,
-              curve: Curves.easeInOut,
-            ),
-        [controller]);
-
-    final colorTween = useMemoized(
-        () => primaryColors.map((color) {
-              return ColorTween(
-                begin: color,
-                end: color,
-              );
-            }).toList(),
-        [primaryColors]);
-    final colors = useMemoized(
-        () => colorTween.map((color) {
-              return color.evaluate(animation)!;
-            }).toList(),
-        [colorTween, animation]);
-
-    final begin = useMemoized(
-        () => AlignmentTween(
-              begin: primaryBegin ?? Alignment.topLeft,
-              end: primaryEnd ?? Alignment.topRight,
-            ),
-        [primaryBegin, primaryEnd]);
-
-    final end = useMemoized(
-        () => AlignmentTween(
-              begin: secondaryBegin ?? Alignment.bottomLeft,
-              end: secondaryEnd ?? Alignment.bottomRight,
-            ),
-        [secondaryBegin, secondaryEnd]);
-
-    return AnimatedBuilder(
-      animation: animation,
-      child: useMemoized(() => child, [child]),
-      builder: (BuildContext context, Widget? child) {
-        return Container(
-          decoration: BoxDecoration(
-            gradient: LinearGradient(
-              begin: animateAlignments
-                  ? begin.evaluate(animation)
-                  : (primaryBegin as Alignment),
-              end: animateAlignments
-                  ? end.evaluate(animation)
-                  : primaryEnd as Alignment,
-              colors: colors,
-            ),
-          ),
-          child: child,
-        );
-      },
-    );
-  }
-}

lib/components/spotube_page_route.dart

@@ -1,24 +0,0 @@
-import 'package:shadcn_flutter/shadcn_flutter.dart';
-
-class SpotubePage<T> extends MaterialPage<T> {
-  const SpotubePage({required super.child});
-}
-
-// class SpotubeSlidePage extends CustomTransitionPage {
-//   SpotubeSlidePage({
-//     required super.child,
-//     super.key,
-//   }) : super(
-//           reverseTransitionDuration: const Duration(milliseconds: 150),
-//           transitionDuration: const Duration(milliseconds: 150),
-//           transitionsBuilder: (context, animation, secondaryAnimation, child) {
-//             return SlideTransition(
-//               position: Tween<Offset>(
-//                 begin: const Offset(1, 0),
-//                 end: Offset.zero,
-//               ).animate(animation),
-//               child: child,
-//             );
-//           },
-//         );
-// }

lib/modules/home/sections/recent.dart

@@ -9,7 +9,7 @@ import 'package:spotube/provider/history/recent.dart';
 
 class HomeRecentlyPlayedSection extends HookConsumerWidget {
   const HomeRecentlyPlayedSection({super.key});
-
+    
   @override
   Widget build(BuildContext context, ref) {
     final history = ref.watch(recentlyPlayedItems);
@@ -20,17 +20,20 @@ class HomeRecentlyPlayedSection extends HookConsumerWidget {
       return const SizedBox();
     }
 
+    final uniqueItems = <dynamic>{};
+    final filteredItems = [
+      for (final item in historyData)
+        if (item.playlist != null && item.playlist?.id != null && uniqueItems.add(item.playlist!.id!))
+          item.playlist
+        else if (item.album != null && item.album?.id != null && uniqueItems.add(item.album?.id))
+          item.album
+    ];
+
     return Skeletonizer(
       enabled: history.isLoading,
       child: HorizontalPlaybuttonCardView(
         title: Text(context.l10n.recently_played),
-        items: [
-          for (final item in historyData)
-            if (item.playlist != null)
-              item.playlist
-            else if (item.album != null)
-              item.album
-        ],
+        items: filteredItems,
         hasNextPage: false,
         isLoadingNextPage: false,
         onFetchMore: () {},

lib/pages/getting_started/sections/region.dart

@@ -133,37 +133,38 @@ class GettingStartedPageLanguageRegionSection extends HookConsumerWidget {
                       popup: SelectPopup.builder(
                         searchPlaceholder: Text(context.l10n.search),
                         builder: (context, searchQuery) {
-                          final filteredLocale = searchQuery?.isNotEmpty != true
-                              ? L10n.all
+                          final hasNotQueried =
+                              searchQuery == null || searchQuery.trim().isEmpty;
+                          final filteredLocale = hasNotQueried
+                              ? [
+                                  const Locale("system", "system"),
+                                  ...L10n.all,
+                                ]
                               : L10n.all
                                   .where(
-                                    (element) =>
-                                        filterLocale(element, searchQuery!),
+                                    (element) => filterLocale(
+                                      element,
+                                      searchQuery.trim(),
+                                    ),
                                   )
                                   .toList();
 
                           return SelectItemBuilder(
-                            childCount: filteredLocale.length + 1,
+                            childCount: filteredLocale.length,
                             builder: (context, index) {
-                              if (index == 0 &&
-                                  searchQuery?.isNotEmpty != true) {
+                              final locale = filteredLocale[index];
+                              if (locale == const Locale("system", "system")) {
                                 return SelectItemButton(
-                                  value: const Locale("system", "system"),
+                                  value: locale,
                                   child: Text(context.l10n.system_default),
                                 );
                               }
-
-                              final indexThen = searchQuery?.isNotEmpty != true
-                                  ? index
-                                  : index - 1;
-
-                              final locale = filteredLocale[indexThen];
                               return SelectItemButton(
                                 value: locale,
                                 child: Text(
                                   LanguageLocals.getDisplayLanguage(
-                                          locale.languageCode)
-                                      .toString(),
+                                    locale.languageCode,
+                                  ).toString(),
                                 ),
                               );
                             },