From c1aa0eadf4f51dc49207db5ba1cb2e94dcfb8ca0 Mon Sep 17 00:00:00 2001 From: Paul Buetow Date: Fri, 22 May 2026 09:39:36 +0300 Subject: Fix final review nits for Material 3 theming (task fb) - Remove redundant multiSelectionEnabled: false from SegmentedButton - DRY up theme toggle tests via optional themeNotifier param in _pumpSettingsScreen - Add UI assertion to verify button reflects ThemeMode.dark after tap Co-Authored-By: Claude Sonnet 4.6 --- player-android/lib/screens/settings_screen.dart | 2 -- 1 file changed, 2 deletions(-) (limited to 'player-android/lib') diff --git a/player-android/lib/screens/settings_screen.dart b/player-android/lib/screens/settings_screen.dart index f7b38bf..53e171a 100644 --- a/player-android/lib/screens/settings_screen.dart +++ b/player-android/lib/screens/settings_screen.dart @@ -293,8 +293,6 @@ class _ThemeToggle extends ConsumerWidget { ), ], selected: {current}, - // Allow only single selection — the user always has exactly one mode active. - multiSelectionEnabled: false, onSelectionChanged: (selection) { // emptySelectionAllowed defaults to false, but guard defensively against future API changes. if (selection.isNotEmpty) { -- cgit v1.2.3