From e8050a78df283bf6edf56fc4071f4d8b7f07e1f5 Mon Sep 17 00:00:00 2001
From: Thibault Deckers <thibault.deckers@gmail.com>
Date: Wed, 23 Jun 2021 12:00:38 +0900
Subject: [PATCH] minor fix

---
 lib/theme/themes.dart                                  | 3 +++
 lib/widgets/dialogs/video_stream_selection_dialog.dart | 4 ++++
 2 files changed, 7 insertions(+)

diff --git a/lib/theme/themes.dart b/lib/theme/themes.dart
index 9062cdf99..df131dcdb 100644
--- a/lib/theme/themes.dart
+++ b/lib/theme/themes.dart
@@ -9,6 +9,9 @@ class Themes {
   static final darkTheme = ThemeData(
     brightness: Brightness.dark,
     accentColor: _accentColor,
+    // canvas color is used as background for the drawer and popups
+    // when using a popup menu on a dialog, lighten the background via `PopupMenuTheme`
+    canvasColor: Colors.grey[850],
     scaffoldBackgroundColor: Colors.grey.shade900,
     dialogBackgroundColor: Colors.grey[850],
     toggleableActiveColor: _accentColor,
diff --git a/lib/widgets/dialogs/video_stream_selection_dialog.dart b/lib/widgets/dialogs/video_stream_selection_dialog.dart
index f4109cdf6..cbe9dfb08 100644
--- a/lib/widgets/dialogs/video_stream_selection_dialog.dart
+++ b/lib/widgets/dialogs/video_stream_selection_dialog.dart
@@ -167,6 +167,10 @@ class _VideoStreamSelectionDialogState extends State<VideoStreamSelectionDialog>
           value: current,
           onChanged: streams.length > 1 ? (newValue) => setState(() => setter(newValue)) : null,
           isExpanded: true,
+          // use a different shade to avoid having the same background
+          // on the dialog (using the theme `dialogBackgroundColor`)
+          // and on the dropdown (using the theme `canvasColor`)
+          dropdownColor: Colors.grey.shade800,
         ),
       ),
     ];