diff options
| author | Paul Buetow <paul@buetow.org> | 2026-05-22 09:39:36 +0300 |
|---|---|---|
| committer | Paul Buetow <paul@buetow.org> | 2026-05-22 09:39:36 +0300 |
| commit | c1aa0eadf4f51dc49207db5ba1cb2e94dcfb8ca0 (patch) | |
| tree | eb0b75610c60946b1893a780aa61a211d30e2205 /player-android/lib | |
| parent | bcf89f586c6dd80f240ac3fb80d01ee3c5f6bcd0 (diff) | |
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 <noreply@anthropic.com>
Diffstat (limited to 'player-android/lib')
| -rw-r--r-- | player-android/lib/screens/settings_screen.dart | 2 |
1 files changed, 0 insertions, 2 deletions
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) { |