From 93d196572f1e68be611cc29f3e05697c848e6cb8 Mon Sep 17 00:00:00 2001
From: edde746 <86283021+edde746@users.noreply.github.com>
Date: Tue, 21 Apr 2026 12:36:15 +0200
Subject: [PATCH] fix: clamp playlist focus index on remove

---
 lib/screens/playlist/playlist_detail_screen.dart | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/lib/screens/playlist/playlist_detail_screen.dart b/lib/screens/playlist/playlist_detail_screen.dart
index ed7ef66b..2be184e5 100644
--- a/lib/screens/playlist/playlist_detail_screen.dart
+++ b/lib/screens/playlist/playlist_detail_screen.dart
@@ -356,6 +356,7 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
   }
 
   Future<void> _removeItem(int index) async {
+    if (items.isEmpty || index < 0 || index >= items.length) return;
     final item = items[index];
 
     // Check if item has playlistItemID (required for removal)
@@ -372,6 +373,12 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
     // Optimistically update UI
     setState(() {
       items.removeAt(index);
+      if (_focusedIndex >= items.length) {
+        _focusedIndex = (items.length - 1).clamp(0, items.length);
+      }
+      if (items.isEmpty) {
+        _focusedColumn = 0;
+      }
     });
 
     // Call API to persist the change
@@ -388,6 +395,7 @@ class _PlaylistDetailScreenState extends BaseMediaListDetailScreen<PlaylistDetai
         appLogger.e('Failed to remove playlist item, reverting UI');
         setState(() {
           items.insert(index, item);
+          _focusedIndex = index;
         });
 
         showErrorSnackBar(context, t.playlists.errorRemoving);