player: no title, better fallback color (#407)

ubuntu-flutter-community · Dec 30, 2023 · 3ce073b · 3ce073b
1 parent 416766c
commit 3ce073b
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 10 deletions.
diff --git a/lib/src/player/bottom_player.dart b/lib/src/player/bottom_player.dart
@@ -19,7 +19,6 @@ class BottomPlayer extends StatelessWidget {
     required this.setFullScreen,
     required this.audio,
     required this.width,
-    this.color,
     required this.playPrevious,
     required this.playNext,
     required this.liked,
@@ -36,7 +35,6 @@ class BottomPlayer extends StatelessWidget {
 
   final Audio? audio;
   final double width;
-  final Color? color;
 
   final Future<void> Function() playPrevious;
   final Future<void> Function() playNext;

diff --git a/lib/src/player/full_height_player.dart b/lib/src/player/full_height_player.dart
@@ -25,7 +25,6 @@ class FullHeightPlayer extends StatelessWidget {
     required this.addStarredStation,
     required this.removeLikedAudio,
     required this.addLikedAudio,
-    this.color,
     required this.setFullScreen,
     required this.playerViewMode,
     required this.onTextTap,
@@ -48,8 +47,6 @@ class FullHeightPlayer extends StatelessWidget {
   final void Function(Audio audio, bool notify) removeLikedAudio;
   final void Function(Audio audio, bool notify) addLikedAudio;
 
-  final Color? color;
-
   final void Function(bool?) setFullScreen;
 
   final PlayerViewMode playerViewMode;
@@ -195,8 +192,8 @@ class FullHeightPlayer extends StatelessWidget {
       children: [
         if (!isMobile)
           HeaderBar(
-            title: Text(
-              audio?.title ?? '',
+            title: const Text(
+              '',
               maxLines: 1,
               overflow: TextOverflow.ellipsis,
             ),

diff --git a/lib/src/player/player_view.dart b/lib/src/player/player_view.dart
@@ -63,7 +63,7 @@ class _PlayerViewState extends State<PlayerView> {
     final c = context.select((PlayerModel m) => m.color);
     final color = getPlayerBg(
       c,
-      theme.isLight ? kCardColorLight : theme.scaffoldBackgroundColor,
+      theme.isLight ? kCardColorLight : kCardColorDark,
     );
     final setFullScreen = playerModel.setFullScreen;
     final playPrevious = playerModel.playPrevious;
@@ -91,7 +91,6 @@ class _PlayerViewState extends State<PlayerView> {
         setFullScreen: setFullScreen,
         nextAudio: nextAudio,
         audio: audio,
-        color: color,
         playPrevious: playPrevious,
         playNext: playNext,
         liked: liked,
@@ -115,7 +114,6 @@ class _PlayerViewState extends State<PlayerView> {
             setFullScreen: setFullScreen,
             audio: audio,
             width: width,
-            color: color,
             playPrevious: playPrevious,
             playNext: playNext,
             liked: liked,