refactor: eliminate duplicate logic across detail screens and helpers

This commit is contained in:
edde746
2026-02-04 09:46:45 +01:00
parent 96c6462d13
commit 16884a7bca
11 changed files with 218 additions and 409 deletions
+20 -6
View File
@@ -17,6 +17,25 @@ class PlexMediaInfo {
int? getPartId() => partId; int? getPartId() => partId;
} }
/// Builds a track label from parts with the standard `' · '` joiner pattern.
///
/// Shared by both Plex track models and MPV track label utilities.
/// If [title] is non-empty it is added first, then [language], then [extraParts].
/// Falls back to `'$fallbackPrefix ${index + 1}'` when no parts are available.
String buildTrackLabel({
String? title,
String? language,
List<String> extraParts = const [],
required int index,
String fallbackPrefix = 'Track',
}) {
final parts = <String>[];
if (title != null && title.isNotEmpty) parts.add(title);
if (language != null && language.isNotEmpty) parts.add(language);
parts.addAll(extraParts);
return parts.isEmpty ? '$fallbackPrefix ${index + 1}' : parts.join(' · ');
}
/// Mixin for building track labels with a consistent pattern /// Mixin for building track labels with a consistent pattern
mixin TrackLabelBuilder { mixin TrackLabelBuilder {
int get id; int get id;
@@ -31,12 +50,7 @@ mixin TrackLabelBuilder {
if (displayTitle != null && displayTitle!.isNotEmpty) { if (displayTitle != null && displayTitle!.isNotEmpty) {
return displayTitle!; return displayTitle!;
} }
final parts = <String>[]; return buildTrackLabel(language: language, extraParts: additionalParts, index: (index ?? id) - 1);
if (language != null && language!.isNotEmpty) {
parts.add(language!);
}
parts.addAll(additionalParts);
return parts.isEmpty ? 'Track ${index ?? id}' : parts.join(' · ');
} }
} }
@@ -9,6 +9,7 @@ import '../utils/provider_extensions.dart';
import '../services/play_queue_launcher.dart'; import '../services/play_queue_launcher.dart';
import '../models/plex_playlist.dart'; import '../models/plex_playlist.dart';
import '../utils/app_logger.dart'; import '../utils/app_logger.dart';
import '../utils/snackbar_helper.dart';
import '../mixins/refreshable.dart'; import '../mixins/refreshable.dart';
import '../mixins/item_updatable.dart'; import '../mixins/item_updatable.dart';
import '../i18n/strings.g.dart'; import '../i18n/strings.g.dart';
@@ -86,7 +87,7 @@ abstract class BaseMediaListDetailScreen<T extends StatefulWidget> extends State
Future<void> _playWithShuffle(bool shuffle) async { Future<void> _playWithShuffle(bool shuffle) async {
if (items.isEmpty) { if (items.isEmpty) {
if (mounted) { if (mounted) {
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(emptyMessage))); showAppSnackBar(context, emptyMessage);
} }
return; return;
} }
+7 -46
View File
@@ -1,12 +1,7 @@
import 'package:flutter/material.dart'; import 'package:flutter/material.dart';
import 'package:provider/provider.dart';
import 'package:material_symbols_icons/symbols.dart'; import 'package:material_symbols_icons/symbols.dart';
import '../models/plex_metadata.dart'; import '../models/plex_metadata.dart';
import '../widgets/focusable_media_card.dart';
import '../widgets/media_grid_delegate.dart';
import '../utils/grid_size_calculator.dart';
import '../widgets/desktop_app_bar.dart'; import '../widgets/desktop_app_bar.dart';
import '../providers/settings_provider.dart';
import '../i18n/strings.g.dart'; import '../i18n/strings.g.dart';
import '../utils/dialogs.dart'; import '../utils/dialogs.dart';
import '../utils/app_logger.dart'; import '../utils/app_logger.dart';
@@ -149,50 +144,16 @@ class _CollectionDetailScreenState extends BaseMediaListDetailScreen<CollectionD
slivers: [ slivers: [
CustomAppBar(title: Text(widget.collection.title), actions: buildFocusableAppBarActions()), CustomAppBar(title: Text(widget.collection.title), actions: buildFocusableAppBarActions()),
...buildStateSlivers(), ...buildStateSlivers(),
if (items.isNotEmpty) _buildFocusableGrid(), if (items.isNotEmpty)
buildFocusableGrid(
items: items,
onRefresh: updateItem,
collectionId: widget.collection.ratingKey,
onListRefresh: loadItems,
),
], ],
), ),
), ),
); );
} }
Widget _buildFocusableGrid() {
return Consumer<SettingsProvider>(
builder: (context, settingsProvider, child) {
final maxExtent = GridSizeCalculator.getMaxCrossAxisExtent(context, settingsProvider.libraryDensity);
return SliverPadding(
padding: const EdgeInsets.fromLTRB(8, 8, 8, 8),
sliver: SliverLayoutBuilder(
builder: (context, constraints) {
final columnCount = GridSizeCalculator.getColumnCount(constraints.crossAxisExtent, maxExtent);
return SliverGrid.builder(
gridDelegate: MediaGridDelegate.createDelegate(
context: context,
density: settingsProvider.libraryDensity,
),
itemCount: items.length,
itemBuilder: (context, index) {
final item = items[index];
final inFirstRow = GridSizeCalculator.isFirstRow(index, columnCount);
final focusNode = index == 0 ? firstItemFocusNode : getGridItemFocusNode(index);
return FocusableMediaCard(
key: Key(item.ratingKey),
item: item,
focusNode: focusNode,
onRefresh: updateItem,
collectionId: widget.collection.ratingKey,
onListRefresh: loadItems,
onNavigateUp: inFirstRow ? navigateToAppBar : null,
onBack: handleBackFromContent,
onFocusChange: (hasFocus) => trackGridItemFocus(index, hasFocus),
);
},
);
},
),
);
},
);
}
} }
@@ -1,9 +1,14 @@
import 'package:flutter/material.dart'; import 'package:flutter/material.dart';
import 'package:flutter/services.dart'; import 'package:flutter/services.dart';
import 'package:provider/provider.dart';
import '../focus/dpad_navigator.dart'; import '../focus/dpad_navigator.dart';
import '../focus/input_mode_tracker.dart'; import '../focus/input_mode_tracker.dart';
import '../focus/key_event_utils.dart'; import '../focus/key_event_utils.dart';
import '../providers/settings_provider.dart';
import '../utils/grid_size_calculator.dart';
import '../widgets/app_icon.dart'; import '../widgets/app_icon.dart';
import '../widgets/focusable_media_card.dart';
import '../widgets/media_grid_delegate.dart';
/// Configuration for app bar buttons /// Configuration for app bar buttons
class AppBarButtonConfig { class AppBarButtonConfig {
@@ -244,4 +249,51 @@ mixin FocusableDetailScreenMixin<T extends StatefulWidget> on State<T> {
lastFocusedContentVersion = contentVersion; lastFocusedContentVersion = contentVersion;
} }
} }
/// Build a standard focusable grid sliver for media items.
/// Used by collection and smart playlist detail screens.
Widget buildFocusableGrid({
required List<dynamic> items,
required void Function(String ratingKey) onRefresh,
String? collectionId,
VoidCallback? onListRefresh,
}) {
return Consumer<SettingsProvider>(
builder: (context, settingsProvider, child) {
final maxExtent = GridSizeCalculator.getMaxCrossAxisExtent(context, settingsProvider.libraryDensity);
return SliverPadding(
padding: const EdgeInsets.fromLTRB(8, 8, 8, 8),
sliver: SliverLayoutBuilder(
builder: (context, constraints) {
final columnCount = GridSizeCalculator.getColumnCount(constraints.crossAxisExtent, maxExtent);
return SliverGrid.builder(
gridDelegate: MediaGridDelegate.createDelegate(
context: context,
density: settingsProvider.libraryDensity,
),
itemCount: items.length,
itemBuilder: (context, index) {
final item = items[index];
final inFirstRow = GridSizeCalculator.isFirstRow(index, columnCount);
final focusNode = index == 0 ? firstItemFocusNode : getGridItemFocusNode(index);
return FocusableMediaCard(
key: Key(item.ratingKey),
item: item,
focusNode: focusNode,
onRefresh: onRefresh,
collectionId: collectionId,
onListRefresh: onListRefresh,
onNavigateUp: inFirstRow ? navigateToAppBar : null,
onBack: handleBackFromContent,
onFocusChange: (hasFocus) => trackGridItemFocus(index, hasFocus),
);
},
);
},
),
);
},
);
}
} }
+99 -300
View File
@@ -1,26 +1,23 @@
import 'package:flutter/material.dart'; import 'package:flutter/material.dart';
import 'package:flutter/services.dart'; import 'package:flutter/services.dart';
import 'package:plezy/widgets/app_icon.dart';
import 'package:material_symbols_icons/symbols.dart'; import 'package:material_symbols_icons/symbols.dart';
import 'package:provider/provider.dart';
import '../../services/plex_client.dart'; import '../../services/plex_client.dart';
import '../../services/play_queue_launcher.dart'; import '../../services/play_queue_launcher.dart';
import '../../models/plex_playlist.dart'; import '../../models/plex_playlist.dart';
import '../../models/plex_metadata.dart'; import '../../models/plex_metadata.dart';
import '../../utils/app_logger.dart'; import '../../utils/app_logger.dart';
import '../../utils/provider_extensions.dart'; import '../../utils/provider_extensions.dart';
import '../../widgets/focusable_media_card.dart'; import '../../widgets/app_icon.dart';
import '../../widgets/media_grid_delegate.dart';
import '../../utils/grid_size_calculator.dart';
import '../../widgets/desktop_app_bar.dart'; import '../../widgets/desktop_app_bar.dart';
import '../../providers/settings_provider.dart';
import '../../focus/dpad_navigator.dart'; import '../../focus/dpad_navigator.dart';
import '../../focus/input_mode_tracker.dart'; import '../../focus/input_mode_tracker.dart';
import '../../focus/key_event_utils.dart'; import '../../focus/key_event_utils.dart';
import 'playlist_item_card.dart'; import 'playlist_item_card.dart';
import '../../i18n/strings.g.dart'; import '../../i18n/strings.g.dart';
import '../../utils/dialogs.dart'; import '../../utils/dialogs.dart';
import '../../utils/snackbar_helper.dart';
import '../base_media_list_detail_screen.dart'; import '../base_media_list_detail_screen.dart';
import '../focusable_detail_screen_mixin.dart';
/// Screen to display the contents of a playlist /// Screen to display the contents of a playlist
class PlaylistDetailScreen extends StatefulWidget { class PlaylistDetailScreen extends StatefulWidget {
@@ -33,7 +30,7 @@ class PlaylistDetailScreen extends StatefulWidget {
} }
class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetailScreen> class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetailScreen>
with StandardItemLoader<PlaylistDetailScreen> { with StandardItemLoader<PlaylistDetailScreen>, FocusableDetailScreenMixin<PlaylistDetailScreen> {
@override @override
dynamic get mediaItem => widget.playlist; dynamic get mediaItem => widget.playlist;
@@ -46,14 +43,41 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
@override @override
IconData get emptyIcon => Symbols.playlist_play_rounded; IconData get emptyIcon => Symbols.playlist_play_rounded;
// Scroll controller for scrolling to top when app bar is focused @override
final ScrollController _scrollController = ScrollController(); bool get hasItems => items.isNotEmpty;
// Focus management @override
int get appBarButtonCount {
int count = 0;
if (items.isNotEmpty) count += 2; // play + shuffle
if (!widget.playlist.smart) count += 1; // delete
return count;
}
@override
List<AppBarButtonConfig> getAppBarButtons() {
final buttons = <AppBarButtonConfig>[];
if (items.isNotEmpty) {
buttons.add(AppBarButtonConfig(icon: Symbols.play_arrow_rounded, tooltip: t.discover.play, onPressed: playItems));
buttons.add(
AppBarButtonConfig(icon: Symbols.shuffle_rounded, tooltip: t.common.shuffle, onPressed: shufflePlayItems),
);
}
if (!widget.playlist.smart) {
buttons.add(
AppBarButtonConfig(
icon: Symbols.delete_rounded,
tooltip: t.playlists.delete,
onPressed: _deletePlaylist,
color: Colors.red,
),
);
}
return buttons;
}
// Focus management for regular (non-smart) reorderable lists
final FocusNode _listFocusNode = FocusNode(debugLabel: 'playlist_list'); final FocusNode _listFocusNode = FocusNode(debugLabel: 'playlist_list');
final FocusNode _playButtonFocusNode = FocusNode(debugLabel: 'playlist_play');
final FocusNode _shuffleButtonFocusNode = FocusNode(debugLabel: 'playlist_shuffle');
final FocusNode _deleteButtonFocusNode = FocusNode(debugLabel: 'playlist_delete');
// Navigation state for regular (non-smart) playlists // Navigation state for regular (non-smart) playlists
int _focusedIndex = 0; int _focusedIndex = 0;
@@ -67,40 +91,13 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
// Estimated item height for scroll-into-view (card + vertical margins) // Estimated item height for scroll-into-view (card + vertical margins)
static const double _estimatedItemHeight = 114.0; static const double _estimatedItemHeight = 114.0;
// App bar focus state
bool _isAppBarFocused = false;
int _appBarFocusedButton = 0; // 0=play, 1=shuffle, 2=delete
// Flag to prevent PopScope from exiting when BACK was handled by a key handler
bool _backHandledByKeyEvent = false;
// Grid focus for smart playlists
final FocusNode _firstItemFocusNode = FocusNode(debugLabel: 'playlist_first_item');
final Map<int, FocusNode> _gridItemFocusNodes = {};
int? _lastFocusedGridIndex;
int _contentVersion = 0;
int _lastFocusedContentVersion = 0;
@override @override
void dispose() { void dispose() {
_scrollController.dispose();
_listFocusNode.dispose(); _listFocusNode.dispose();
_playButtonFocusNode.dispose(); disposeFocusResources();
_shuffleButtonFocusNode.dispose();
_deleteButtonFocusNode.dispose();
_firstItemFocusNode.dispose();
for (final node in _gridItemFocusNodes.values) {
node.dispose();
}
_gridItemFocusNodes.clear();
super.dispose(); super.dispose();
} }
/// Get or create a focus node for a grid item at the given index
FocusNode _getGridItemFocusNode(int index) {
return _gridItemFocusNodes.putIfAbsent(index, () => FocusNode(debugLabel: 'playlist_grid_item_$index'));
}
@override @override
Future<List<PlexMetadata>> fetchItems() async { Future<List<PlexMetadata>> fetchItems() async {
return await client.getPlaylist(widget.playlist.ratingKey); return await client.getPlaylist(widget.playlist.ratingKey);
@@ -109,7 +106,7 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
@override @override
Future<void> loadItems() async { Future<void> loadItems() async {
// Increment content version when loading fresh content // Increment content version when loading fresh content
_contentVersion++; contentVersion++;
await super.loadItems(); await super.loadItems();
// Auto-focus after load if in keyboard mode // Auto-focus after load if in keyboard mode
@@ -118,12 +115,12 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
if (!mounted) return; if (!mounted) return;
if (InputModeTracker.isKeyboardMode(context)) { if (InputModeTracker.isKeyboardMode(context)) {
setState(() { setState(() {
_isAppBarFocused = false; isAppBarFocused = false;
_focusedIndex = 0; _focusedIndex = 0;
_focusedColumn = 0; _focusedColumn = 0;
}); });
if (widget.playlist.smart) { if (widget.playlist.smart) {
_firstItemFocusNode.requestFocus(); firstItemFocusNode.requestFocus();
} else { } else {
_listFocusNode.requestFocus(); _listFocusNode.requestFocus();
} }
@@ -137,6 +134,21 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
return 'Loaded $itemCount items for playlist: ${widget.playlist.title}'; return 'Loaded $itemCount items for playlist: ${widget.playlist.title}';
} }
/// Navigate from app bar down to content - overridden to handle both grid and list
@override
void navigateToGrid() {
if (!hasItems) return;
if (widget.playlist.smart) {
super.navigateToGrid();
} else {
setState(() {
isAppBarFocused = false;
});
_listFocusNode.requestFocus();
}
}
/// Get the correct PlexClient for this playlist's server /// Get the correct PlexClient for this playlist's server
PlexClient _getClientForPlaylist() { PlexClient _getClientForPlaylist() {
return context.getClientForServer(widget.playlist.serverId!); return context.getClientForServer(widget.playlist.serverId!);
@@ -154,10 +166,10 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
if (mounted) { if (mounted) {
if (success) { if (success) {
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.playlists.deleted))); showSuccessSnackBar(context, t.playlists.deleted);
Navigator.pop(context); // Return to playlists screen Navigator.pop(context); // Return to playlists screen
} else { } else {
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.playlists.errorDeleting))); showErrorSnackBar(context, t.playlists.errorDeleting);
} }
} }
} }
@@ -178,7 +190,7 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
if (movedItem.playlistItemID == null) { if (movedItem.playlistItemID == null) {
appLogger.e('Cannot reorder: item missing playlistItemID'); appLogger.e('Cannot reorder: item missing playlistItemID');
if (mounted) { if (mounted) {
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.playlists.errorReordering))); showErrorSnackBar(context, t.playlists.errorReordering);
} }
return; return;
} }
@@ -194,7 +206,7 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
if (afterItem.playlistItemID == null) { if (afterItem.playlistItemID == null) {
appLogger.e('Cannot reorder: after item missing playlistItemID'); appLogger.e('Cannot reorder: after item missing playlistItemID');
if (mounted) { if (mounted) {
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.playlists.errorReordering))); showErrorSnackBar(context, t.playlists.errorReordering);
} }
return; return;
} }
@@ -225,7 +237,7 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
items.insert(oldIndex, item); items.insert(oldIndex, item);
}); });
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.playlists.errorReordering))); showErrorSnackBar(context, t.playlists.errorReordering);
} }
} }
} }
@@ -240,7 +252,7 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
if (movedItem.playlistItemID == null) { if (movedItem.playlistItemID == null) {
appLogger.e('Cannot persist move: item missing playlistItemID'); appLogger.e('Cannot persist move: item missing playlistItemID');
if (mounted) { if (mounted) {
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.playlists.errorReordering))); showErrorSnackBar(context, t.playlists.errorReordering);
// Revert the UI change // Revert the UI change
setState(() { setState(() {
final item = items.removeAt(newIndex); final item = items.removeAt(newIndex);
@@ -260,7 +272,7 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
if (afterItem.playlistItemID == null) { if (afterItem.playlistItemID == null) {
appLogger.e('Cannot persist move: after item missing playlistItemID'); appLogger.e('Cannot persist move: after item missing playlistItemID');
if (mounted) { if (mounted) {
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.playlists.errorReordering))); showErrorSnackBar(context, t.playlists.errorReordering);
// Revert the UI change // Revert the UI change
setState(() { setState(() {
final item = items.removeAt(newIndex); final item = items.removeAt(newIndex);
@@ -291,7 +303,7 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
items.insert(originalIndex, item); items.insert(originalIndex, item);
_focusedIndex = originalIndex; _focusedIndex = originalIndex;
}); });
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.playlists.errorReordering))); showErrorSnackBar(context, t.playlists.errorReordering);
} }
} }
} }
@@ -303,7 +315,7 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
if (item.playlistItemID == null) { if (item.playlistItemID == null) {
appLogger.e('Cannot remove: item missing playlistItemID'); appLogger.e('Cannot remove: item missing playlistItemID');
if (mounted) { if (mounted) {
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.playlists.errorRemoving))); showErrorSnackBar(context, t.playlists.errorRemoving);
} }
return; return;
} }
@@ -323,7 +335,7 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
if (mounted) { if (mounted) {
if (success) { if (success) {
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.playlists.itemRemoved))); showSuccessSnackBar(context, t.playlists.itemRemoved);
} else { } else {
// Revert on failure // Revert on failure
appLogger.e('Failed to remove playlist item, reverting UI'); appLogger.e('Failed to remove playlist item, reverting UI');
@@ -331,7 +343,7 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
items.insert(index, item); items.insert(index, item);
}); });
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.playlists.errorRemoving))); showErrorSnackBar(context, t.playlists.errorRemoving);
} }
} }
} }
@@ -360,19 +372,19 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
/// Uses estimated item height instead of per-item GlobalKeys. /// Uses estimated item height instead of per-item GlobalKeys.
void _ensureFocusedVisible() { void _ensureFocusedVisible() {
WidgetsBinding.instance.addPostFrameCallback((_) { WidgetsBinding.instance.addPostFrameCallback((_) {
if (!mounted || !_scrollController.hasClients) return; if (!mounted || !scrollController.hasClients) return;
final targetOffset = _focusedIndex * _estimatedItemHeight; final targetOffset = _focusedIndex * _estimatedItemHeight;
final viewportHeight = _scrollController.position.viewportDimension; final viewportHeight = scrollController.position.viewportDimension;
final currentOffset = _scrollController.offset; final currentOffset = scrollController.offset;
// Check if the item is outside the visible area (with some padding) // Check if the item is outside the visible area (with some padding)
if (targetOffset < currentOffset || targetOffset > currentOffset + viewportHeight - _estimatedItemHeight) { if (targetOffset < currentOffset || targetOffset > currentOffset + viewportHeight - _estimatedItemHeight) {
// Scroll so the item sits ~25% from the top of the viewport // Scroll so the item sits ~25% from the top of the viewport
final scrollTo = (targetOffset - viewportHeight * 0.25).clamp( final scrollTo = (targetOffset - viewportHeight * 0.25).clamp(
_scrollController.position.minScrollExtent, scrollController.position.minScrollExtent,
_scrollController.position.maxScrollExtent, scrollController.position.maxScrollExtent,
); );
_scrollController.animateTo(scrollTo, duration: const Duration(milliseconds: 200), curve: Curves.easeOut); scrollController.animateTo(scrollTo, duration: const Duration(milliseconds: 200), curve: Curves.easeOut);
} }
}); });
} }
@@ -384,11 +396,11 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
final backResult = handleBackKeyAction(event, () { final backResult = handleBackKeyAction(event, () {
if (_movingIndex != null) { if (_movingIndex != null) {
// Cancel move mode, set flag to prevent PopScope exit // Cancel move mode, set flag to prevent PopScope exit
_backHandledByKeyEvent = true; backHandledByKeyEvent = true;
_cancelMoveMode(); _cancelMoveMode();
} else { } else {
// Navigate to app bar on BACK, set flag to prevent PopScope exit // Navigate to app bar on BACK, set flag to prevent PopScope exit
_handleBackFromContent(); handleBackFromContent();
} }
}); });
if (backResult != KeyEventResult.ignored) { if (backResult != KeyEventResult.ignored) {
@@ -446,7 +458,7 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
_ensureFocusedVisible(); _ensureFocusedVisible();
} else { } else {
// First item - navigate to app bar // First item - navigate to app bar
_navigateToAppBar(); navigateToAppBar();
} }
return KeyEventResult.handled; return KeyEventResult.handled;
} }
@@ -504,192 +516,6 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
return KeyEventResult.ignored; return KeyEventResult.ignored;
} }
/// Handle key events when app bar is focused
KeyEventResult _handleAppBarKeyEvent(FocusNode node, KeyEvent event) {
final key = event.logicalKey;
final hasDelete = !widget.playlist.smart;
final maxButton = hasDelete ? 2 : 1;
final backResult = handleBackKeyAction(event, () => Navigator.pop(context));
if (backResult != KeyEventResult.ignored) {
return backResult;
}
if (event is! KeyDownEvent) return KeyEventResult.ignored;
if (key.isLeftKey && _appBarFocusedButton > 0) {
setState(() => _appBarFocusedButton--);
_focusAppBarButton(_appBarFocusedButton);
return KeyEventResult.handled;
}
if (key.isRightKey && _appBarFocusedButton < maxButton) {
setState(() => _appBarFocusedButton++);
_focusAppBarButton(_appBarFocusedButton);
return KeyEventResult.handled;
}
if (key.isDownKey) {
// Return focus to list/grid
setState(() => _isAppBarFocused = false);
if (items.isNotEmpty) {
if (widget.playlist.smart) {
_navigateToGrid();
} else {
_listFocusNode.requestFocus();
}
}
return KeyEventResult.handled;
}
if (key.isSelectKey) {
_triggerAppBarButton(_appBarFocusedButton);
return KeyEventResult.handled;
}
return KeyEventResult.ignored;
}
void _focusAppBarButton(int index) {
switch (index) {
case 0:
_playButtonFocusNode.requestFocus();
break;
case 1:
_shuffleButtonFocusNode.requestFocus();
break;
case 2:
_deleteButtonFocusNode.requestFocus();
break;
}
}
void _triggerAppBarButton(int index) {
switch (index) {
case 0:
playItems();
break;
case 1:
shufflePlayItems();
break;
case 2:
if (!widget.playlist.smart) _deletePlaylist();
break;
}
}
/// Navigate focus from app bar down to the grid
void _navigateToGrid() {
if (items.isEmpty) return;
// Check if we should restore focus to the last focused item
final shouldRestoreFocus =
_lastFocusedGridIndex != null &&
_lastFocusedContentVersion == _contentVersion &&
_lastFocusedGridIndex! < items.length;
final targetIndex = shouldRestoreFocus ? _lastFocusedGridIndex! : 0;
if (targetIndex == 0) {
_firstItemFocusNode.requestFocus();
} else {
_getGridItemFocusNode(targetIndex).requestFocus();
}
}
/// Navigate from grid to app bar
void _navigateToAppBar() {
setState(() {
_isAppBarFocused = true;
_appBarFocusedButton = 0;
});
_playButtonFocusNode.requestFocus();
// Scroll to top to show the app bar
_scrollController.animateTo(0, duration: const Duration(milliseconds: 200), curve: Curves.easeOut);
}
/// Handle BACK key from grid/list - navigate to app bar and set flag to prevent PopScope exit
void _handleBackFromContent() {
_backHandledByKeyEvent = true;
_navigateToAppBar();
}
/// Handle back navigation for PopScope
bool _handleBackNavigation() {
// If BACK was already handled by a key event, don't pop
if (_backHandledByKeyEvent) {
_backHandledByKeyEvent = false;
return false;
}
// If in move mode, cancel move instead of navigating
if (_movingIndex != null) {
_cancelMoveMode();
return false;
}
if (_isAppBarFocused) {
// Already on app bar, allow exit
return true;
} else {
// Focus app bar first
_navigateToAppBar();
return false;
}
}
/// Build focusable app bar actions
List<Widget> _buildFocusableAppBarActions() {
final colorScheme = Theme.of(context).colorScheme;
final isKeyboardMode = InputModeTracker.isKeyboardMode(context);
Widget buildFocusableButton({
required FocusNode focusNode,
required int buttonIndex,
required IconData icon,
required String tooltip,
required VoidCallback onPressed,
Color? color,
}) {
final isFocused = isKeyboardMode && _isAppBarFocused && _appBarFocusedButton == buttonIndex;
return Focus(
focusNode: focusNode,
onKeyEvent: _handleAppBarKeyEvent,
child: Container(
decoration: isFocused
? BoxDecoration(color: colorScheme.surfaceContainerHighest, borderRadius: BorderRadius.circular(20))
: null,
child: IconButton(icon: AppIcon(icon, fill: 1), tooltip: tooltip, onPressed: onPressed, color: color),
),
);
}
return [
if (items.isNotEmpty)
buildFocusableButton(
focusNode: _playButtonFocusNode,
buttonIndex: 0,
icon: Symbols.play_arrow_rounded,
tooltip: t.discover.play,
onPressed: playItems,
),
if (items.isNotEmpty)
buildFocusableButton(
focusNode: _shuffleButtonFocusNode,
buttonIndex: 1,
icon: Symbols.shuffle_rounded,
tooltip: t.common.shuffle,
onPressed: shufflePlayItems,
),
if (!widget.playlist.smart)
buildFocusableButton(
focusNode: _deleteButtonFocusNode,
buttonIndex: 2,
icon: Symbols.delete_rounded,
tooltip: t.playlists.delete,
onPressed: _deletePlaylist,
color: Colors.red,
),
];
}
/// Cancel move mode if active, returns true if cancelled /// Cancel move mode if active, returns true if cancelled
bool _cancelMoveMode() { bool _cancelMoveMode() {
if (_movingIndex != null) { if (_movingIndex != null) {
@@ -707,6 +533,23 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
return false; return false;
} }
/// Handle back navigation for PopScope - extends mixin with move mode support
bool _handleBackNavigation() {
// If BACK was already handled by a key event, don't pop
if (backHandledByKeyEvent) {
backHandledByKeyEvent = false;
return false;
}
// If in move mode, cancel move instead of navigating
if (_movingIndex != null) {
_cancelMoveMode();
return false;
}
return handleBackNavigation();
}
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
final isKeyboardMode = InputModeTracker.isKeyboardMode(context); final isKeyboardMode = InputModeTracker.isKeyboardMode(context);
@@ -716,7 +559,7 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
final needsListFocus = !widget.playlist.smart && items.isNotEmpty; final needsListFocus = !widget.playlist.smart && items.isNotEmpty;
Widget scrollView = CustomScrollView( Widget scrollView = CustomScrollView(
controller: _scrollController, controller: scrollController,
slivers: [ slivers: [
CustomAppBar( CustomAppBar(
title: Column( title: Column(
@@ -737,13 +580,13 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
), ),
], ],
), ),
actions: _buildFocusableAppBarActions(), actions: buildFocusableAppBarActions(),
), ),
...buildStateSlivers(), ...buildStateSlivers(),
if (items.isNotEmpty) if (items.isNotEmpty)
if (widget.playlist.smart) if (widget.playlist.smart)
// Smart playlists: Use focusable grid view (cannot be reordered) // Smart playlists: Use focusable grid view (cannot be reordered)
_buildSmartPlaylistGrid(isKeyboardMode) buildFocusableGrid(items: items, onRefresh: updateItem)
else else
// Regular playlists: Use sliver reorderable list // Regular playlists: Use sliver reorderable list
_buildReorderableList(isKeyboardMode), _buildReorderableList(isKeyboardMode),
@@ -752,13 +595,13 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
if (needsListFocus) { if (needsListFocus) {
scrollView = Focus( scrollView = Focus(
autofocus: isKeyboardMode && !_isAppBarFocused, autofocus: isKeyboardMode && !isAppBarFocused,
focusNode: _listFocusNode, focusNode: _listFocusNode,
onKeyEvent: _handleListKeyEvent, onKeyEvent: _handleListKeyEvent,
onFocusChange: (hasFocus) { onFocusChange: (hasFocus) {
if (hasFocus && mounted) { if (hasFocus && mounted) {
setState(() { setState(() {
_isAppBarFocused = false; isAppBarFocused = false;
}); });
} }
}, },
@@ -780,50 +623,6 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
); );
} }
/// Build a focusable grid for smart playlists
Widget _buildSmartPlaylistGrid(bool isKeyboardMode) {
return Consumer<SettingsProvider>(
builder: (context, settingsProvider, child) {
final maxExtent = GridSizeCalculator.getMaxCrossAxisExtent(context, settingsProvider.libraryDensity);
return SliverPadding(
padding: const EdgeInsets.fromLTRB(8, 8, 8, 8),
sliver: SliverLayoutBuilder(
builder: (context, constraints) {
final columnCount = GridSizeCalculator.getColumnCount(constraints.crossAxisExtent, maxExtent);
return SliverGrid.builder(
gridDelegate: MediaGridDelegate.createDelegate(
context: context,
density: settingsProvider.libraryDensity,
),
itemCount: items.length,
itemBuilder: (context, index) {
final item = items[index];
final isFirstRow = GridSizeCalculator.isFirstRow(index, columnCount);
final focusNode = index == 0 ? _firstItemFocusNode : _getGridItemFocusNode(index);
return FocusableMediaCard(
key: Key(item.ratingKey),
item: item,
focusNode: focusNode,
onRefresh: updateItem,
onNavigateUp: isFirstRow ? _navigateToAppBar : null,
onBack: _handleBackFromContent,
onFocusChange: (hasFocus) {
if (hasFocus) {
_lastFocusedGridIndex = index;
_lastFocusedContentVersion = _contentVersion;
}
},
);
},
);
},
),
);
},
);
}
/// Build a reorderable list for regular playlists with focus support /// Build a reorderable list for regular playlists with focus support
Widget _buildReorderableList(bool isKeyboardMode) { Widget _buildReorderableList(bool isKeyboardMode) {
return SliverReorderableList( return SliverReorderableList(
@@ -833,7 +632,7 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
final item = items[index]; final item = items[index];
// Check keyboard mode directly to ensure we get latest value // Check keyboard mode directly to ensure we get latest value
final inKeyboardMode = InputModeTracker.isKeyboardMode(context); final inKeyboardMode = InputModeTracker.isKeyboardMode(context);
final isFocused = inKeyboardMode && index == _focusedIndex && !_isAppBarFocused; final isFocused = inKeyboardMode && index == _focusedIndex && !isAppBarFocused;
final isMoving = index == _movingIndex; final isMoving = index == _movingIndex;
return RepaintBoundary( return RepaintBoundary(
+3 -2
View File
@@ -5,6 +5,7 @@ import 'package:flutter/services.dart';
import 'package:logger/logger.dart'; import 'package:logger/logger.dart';
import '../../i18n/strings.g.dart'; import '../../i18n/strings.g.dart';
import '../../utils/app_logger.dart'; import '../../utils/app_logger.dart';
import '../../utils/snackbar_helper.dart';
import '../../widgets/desktop_app_bar.dart'; import '../../widgets/desktop_app_bar.dart';
class LogsScreen extends StatefulWidget { class LogsScreen extends StatefulWidget {
@@ -42,7 +43,7 @@ class _LogsScreenState extends State<LogsScreen> {
MemoryLogOutput.clearLogs(); MemoryLogOutput.clearLogs();
_logs = []; _logs = [];
}); });
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.messages.logsCleared))); showSuccessSnackBar(context, t.messages.logsCleared);
} }
void _copyAllLogs() { void _copyAllLogs() {
@@ -63,7 +64,7 @@ class _LogsScreenState extends State<LogsScreen> {
} }
} }
Clipboard.setData(ClipboardData(text: buffer.toString())); Clipboard.setData(ClipboardData(text: buffer.toString()));
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.messages.logsCopied))); showSuccessSnackBar(context, t.messages.logsCopied);
} }
Color _getLevelColor(Level level) { Color _getLevelColor(Level level) {
+4 -3
View File
@@ -3,6 +3,7 @@ import 'package:plezy/widgets/app_icon.dart';
import 'package:material_symbols_icons/symbols.dart'; import 'package:material_symbols_icons/symbols.dart';
import '../../i18n/strings.g.dart'; import '../../i18n/strings.g.dart';
import '../../models/mpv_config_models.dart'; import '../../models/mpv_config_models.dart';
import '../../utils/snackbar_helper.dart';
import '../../services/settings_service.dart'; import '../../services/settings_service.dart';
import '../../widgets/desktop_app_bar.dart'; import '../../widgets/desktop_app_bar.dart';
@@ -202,7 +203,7 @@ class _MpvConfigScreenState extends State<MpvConfigScreen> {
}); });
if (mounted) { if (mounted) {
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.mpvConfig.presetSaved))); showSuccessSnackBar(context, t.mpvConfig.presetSaved);
} }
} }
@@ -216,7 +217,7 @@ class _MpvConfigScreenState extends State<MpvConfigScreen> {
}); });
if (mounted) { if (mounted) {
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.mpvConfig.presetLoaded))); showAppSnackBar(context, t.mpvConfig.presetLoaded);
} }
} }
@@ -233,7 +234,7 @@ class _MpvConfigScreenState extends State<MpvConfigScreen> {
}); });
if (mounted) { if (mounted) {
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.mpvConfig.presetDeleted))); showSuccessSnackBar(context, t.mpvConfig.presetDeleted);
} }
} }
} }
+9 -21
View File
@@ -1297,11 +1297,10 @@ class _SettingsScreenState extends State<SettingsScreen> with FocusableTab {
TextButton( TextButton(
onPressed: () async { onPressed: () async {
final navigator = Navigator.of(context); final navigator = Navigator.of(context);
final messenger = ScaffoldMessenger.of(context);
await _settingsService.clearCache(); await _settingsService.clearCache();
if (mounted) { if (mounted) {
navigator.pop(); navigator.pop();
messenger.showSnackBar(SnackBar(content: Text(t.settings.clearCacheSuccess))); showSuccessSnackBar(context, t.settings.clearCacheSuccess);
} }
}, },
child: Text(t.common.clear), child: Text(t.common.clear),
@@ -1324,12 +1323,11 @@ class _SettingsScreenState extends State<SettingsScreen> with FocusableTab {
TextButton( TextButton(
onPressed: () async { onPressed: () async {
final navigator = Navigator.of(context); final navigator = Navigator.of(context);
final messenger = ScaffoldMessenger.of(context);
await _settingsService.resetAllSettings(); await _settingsService.resetAllSettings();
await _keyboardService?.resetToDefaults(); await _keyboardService?.resetToDefaults();
if (mounted) { if (mounted) {
navigator.pop(); navigator.pop();
messenger.showSnackBar(SnackBar(content: Text(t.settings.resetSettingsSuccess))); showSuccessSnackBar(context, t.settings.resetSettingsSuccess);
// Reload settings // Reload settings
_loadSettings(); _loadSettings();
} }
@@ -1641,11 +1639,10 @@ class _KeyboardShortcutsScreenState extends State<_KeyboardShortcutsScreen> {
actions: [ actions: [
TextButton( TextButton(
onPressed: () async { onPressed: () async {
final messenger = ScaffoldMessenger.of(context);
await widget.keyboardService.resetToDefaults(); await widget.keyboardService.resetToDefaults();
await _loadHotkeys(); await _loadHotkeys();
if (mounted) { if (mounted) {
messenger.showSnackBar(SnackBar(content: Text(t.settings.shortcutsReset))); showSuccessSnackBar(context, t.settings.shortcutsReset);
} }
}, },
child: Text(t.common.reset), child: Text(t.common.reset),
@@ -1696,20 +1693,14 @@ class _KeyboardShortcutsScreenState extends State<_KeyboardShortcutsScreen> {
currentHotKey: currentHotkey, currentHotKey: currentHotkey,
onHotKeyRecorded: (newHotkey) async { onHotKeyRecorded: (newHotkey) async {
final navigator = Navigator.of(context); final navigator = Navigator.of(context);
final messenger = ScaffoldMessenger.of(context);
// Check for conflicts // Check for conflicts
final existingAction = widget.keyboardService.getActionForHotkey(newHotkey); final existingAction = widget.keyboardService.getActionForHotkey(newHotkey);
if (existingAction != null && existingAction != action) { if (existingAction != null && existingAction != action) {
navigator.pop(); navigator.pop();
messenger.showSnackBar( showErrorSnackBar(
SnackBar( context,
content: Text( t.settings.shortcutAlreadyAssigned(action: widget.keyboardService.getActionDisplayName(existingAction)),
t.settings.shortcutAlreadyAssigned(
action: widget.keyboardService.getActionDisplayName(existingAction),
),
),
),
); );
return; return;
} }
@@ -1725,12 +1716,9 @@ class _KeyboardShortcutsScreenState extends State<_KeyboardShortcutsScreen> {
navigator.pop(); navigator.pop();
messenger.showSnackBar( showSuccessSnackBar(
SnackBar( context,
content: Text( t.settings.shortcutUpdated(action: widget.keyboardService.getActionDisplayName(action)),
t.settings.shortcutUpdated(action: widget.keyboardService.getActionDisplayName(action)),
),
),
); );
} }
}, },
+3 -10
View File
@@ -908,7 +908,7 @@ class VideoPlayerScreenState extends State<VideoPlayerScreen> with WidgetsBindin
} }
} on PlaybackException catch (e) { } on PlaybackException catch (e) {
if (mounted) { if (mounted) {
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(e.message))); showErrorSnackBar(context, e.message);
} }
} catch (e) { } catch (e) {
if (mounted) { if (mounted) {
@@ -1323,12 +1323,7 @@ class VideoPlayerScreenState extends State<VideoPlayerScreen> with WidgetsBindin
appLogger.e('[Player ERROR] $error'); appLogger.e('[Player ERROR] $error');
if (!mounted) return; if (!mounted) return;
ScaffoldMessenger.of(context).showSnackBar( showErrorSnackBar(context, t.messages.failedPlayback(action: 'play', error: error));
SnackBar(
content: Text(t.messages.failedPlayback(action: 'play', error: error)),
duration: const Duration(seconds: 4),
),
);
} }
/// Handle notification when native player switched from ExoPlayer to MPV /// Handle notification when native player switched from ExoPlayer to MPV
@@ -1336,9 +1331,7 @@ class VideoPlayerScreenState extends State<VideoPlayerScreen> with WidgetsBindin
appLogger.i('Player backend switched from ExoPlayer to MPV (native fallback)'); appLogger.i('Player backend switched from ExoPlayer to MPV (native fallback)');
if (mounted) { if (mounted) {
ScaffoldMessenger.of(context).showSnackBar( showAppSnackBar(context, t.messages.switchingToCompatiblePlayer);
SnackBar(content: Text(t.messages.switchingToCompatiblePlayer), duration: const Duration(seconds: 2)),
);
} }
} }
+17 -19
View File
@@ -1,6 +1,10 @@
import 'codec_utils.dart'; import 'codec_utils.dart';
import '../models/plex_media_info.dart' show buildTrackLabel;
/// Utility for building track labels for audio and subtitle tracks. /// Utility for building track labels for audio and subtitle tracks.
///
/// Delegates to the shared [buildTrackLabel] function so Plex-model and
/// MPV-player label logic stays consistent.
class TrackLabelBuilder { class TrackLabelBuilder {
TrackLabelBuilder._(); TrackLabelBuilder._();
@@ -14,36 +18,30 @@ class TrackLabelBuilder {
int? channelsCount, int? channelsCount,
required int index, required int index,
}) { }) {
final parts = <String>[]; final extraParts = <String>[];
if (title != null && title.isNotEmpty) {
parts.add(title);
}
if (language != null && language.isNotEmpty) {
parts.add(language.toUpperCase());
}
if (codec != null && codec.isNotEmpty) { if (codec != null && codec.isNotEmpty) {
parts.add(CodecUtils.formatAudioCodec(codec)); extraParts.add(CodecUtils.formatAudioCodec(codec));
} }
if (channelsCount != null) { if (channelsCount != null) {
parts.add('${channelsCount}ch'); extraParts.add('${channelsCount}ch');
} }
return parts.isEmpty ? 'Audio Track ${index + 1}' : parts.join(' · '); return buildTrackLabel(
title: title,
language: language?.toUpperCase(),
extraParts: extraParts,
index: index,
fallbackPrefix: 'Audio Track',
);
} }
/// Build a label for a subtitle track. /// Build a label for a subtitle track.
/// ///
/// Combines title, language, and codec (with friendly codec names). /// Combines title, language, and codec (with friendly codec names).
static String buildSubtitleLabel({String? title, String? language, String? codec, required int index}) { static String buildSubtitleLabel({String? title, String? language, String? codec, required int index}) {
final parts = <String>[]; final extraParts = <String>[];
if (title != null && title.isNotEmpty) {
parts.add(title);
}
if (language != null && language.isNotEmpty) {
parts.add(language.toUpperCase());
}
if (codec != null && codec.isNotEmpty) { if (codec != null && codec.isNotEmpty) {
parts.add(CodecUtils.formatSubtitleCodec(codec)); extraParts.add(CodecUtils.formatSubtitleCodec(codec));
} }
return parts.isEmpty ? 'Track ${index + 1}' : parts.join(' · '); return buildTrackLabel(title: title, language: language?.toUpperCase(), extraParts: extraParts, index: index);
} }
} }
@@ -4,6 +4,7 @@ import 'package:material_symbols_icons/symbols.dart';
import 'package:provider/provider.dart'; import 'package:provider/provider.dart';
import '../../i18n/strings.g.dart'; import '../../i18n/strings.g.dart';
import '../../utils/snackbar_helper.dart';
import '../models/watch_session.dart'; import '../models/watch_session.dart';
import '../providers/watch_together_provider.dart'; import '../providers/watch_together_provider.dart';
@@ -255,7 +256,7 @@ class _SessionMenuSheet extends StatelessWidget {
void _copySessionCode(BuildContext context, String sessionId) { void _copySessionCode(BuildContext context, String sessionId) {
Clipboard.setData(ClipboardData(text: sessionId)); Clipboard.setData(ClipboardData(text: sessionId));
ScaffoldMessenger.of(context).showSnackBar(SnackBar(content: Text(t.watchTogether.sessionCodeCopied))); showSuccessSnackBar(context, t.watchTogether.sessionCodeCopied);
} }
void _confirmLeave(BuildContext context) { void _confirmLeave(BuildContext context) {