From dfddc49df27bc979598a4f3fff4096cdada430db Mon Sep 17 00:00:00 2001 From: Chaphasilor Date: Mon, 28 Oct 2024 11:58:54 +0100 Subject: [PATCH] remove mini visualizer, tint title - the visualizer doesn't fit in well and isn't needed to mark the current track anymore - the fully-saturated accent color looked a bit strange --- .../AlbumScreen/track_list_tile.dart | 47 +++++++------------ 1 file changed, 16 insertions(+), 31 deletions(-) diff --git a/lib/components/AlbumScreen/track_list_tile.dart b/lib/components/AlbumScreen/track_list_tile.dart index c89a38ad..af0b8397 100644 --- a/lib/components/AlbumScreen/track_list_tile.dart +++ b/lib/components/AlbumScreen/track_list_tile.dart @@ -599,10 +599,18 @@ class ThemedTrackListTile extends ConsumerWidget { .bodyLarge ?.copyWith( color: isCurrentTrack - ? ref + ? Color.alphaBlend( + (ref .watch(colorThemeNullableProvider) .value - ?.secondary + ?.secondary + .withOpacity(0.5)) ?? + Colors.transparent, + Theme.of(context) + .textTheme + .bodyLarge + ?.color ?? + Colors.black) : Theme.of(context).textTheme.bodyLarge?.color, ), )), @@ -704,35 +712,12 @@ class TrackListItemTile extends StatelessWidget { ))), ), if (showCover) - Stack( - children: [ - AlbumImage( - item: baseItem, - borderRadius: isCurrentTrack - ? BorderRadius.zero - : BorderRadius.circular(8.0), - themeCallback: themeCallback, - ), - if (isCurrentTrack) - SizedBox.square( - dimension: defaultTileHeight, - child: Container( - decoration: BoxDecoration( - borderRadius: isCurrentTrack - ? BorderRadius.zero - : BorderRadius.circular(8.0), - color: Theme.of(context).brightness == Brightness.dark - ? Colors.black.withOpacity(0.35) - : Colors.white.withOpacity(0.35), - ), - // color: Theme.of(context).colorScheme.primary.withOpacity(0.35), - child: MiniMusicVisualizer( - color: Theme.of(context).colorScheme.secondary, - animate: true, - ), - ), - ), - ], + AlbumImage( + item: baseItem, + borderRadius: isCurrentTrack + ? BorderRadius.zero + : BorderRadius.circular(8.0), + themeCallback: themeCallback, ), ], ),