Makussu/subtracks
1
0
Fork 0
mirror of https://github.com/austinried/subtracks.git synced 2026-11-11 07:29:14 +01:00
Code Issues Packages Projects Releases Wiki Activity

document why the queue view keeps a per-section reorder state and index-keyed headers

Browse Source
This commit is contained in:
austinried
2026-10-04 00:30:00 +00:00
parent 34f0e15a7a
commit eb5f5703f4
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/src/main/kotlin/com/subtracks/ui/playback/QueueScreen.kt
+6
View File
@@ -327,6 +327,10 @@ fun QueueScreen(
var dragId by remember { mutableStateOf<Long?>(null) } var dragId by remember { mutableStateOf<Long?>(null) }
var dragFrom by remember { mutableStateOf<Long?>(null) } var dragFrom by remember { mutableStateOf<Long?>(null) }
// One reorder state per section: the library only targets keys registered with the state that
// started the drag, so an up-next drag cannot reach the context rows and vice versa.
// scrollThreshold 0 disables auto-scroll, which otherwise keeps scrolling past the section
// looking for a target that does not exist.
val upNextReorder = val upNextReorder =
rememberReorderableLazyListState(listState, scrollThreshold = 0.dp) { from, to -> rememberReorderableLazyListState(listState, scrollThreshold = 0.dp) { from, to ->
onReorder(rows.rowIndexOf(from.key), rows.rowIndexOf(to.key)) onReorder(rows.rowIndexOf(from.key), rows.rowIndexOf(to.key))
@@ -389,6 +393,8 @@ fun QueueScreen(
) { ) {
rows.forEachIndexed { index, row -> rows.forEachIndexed { index, row ->
if (index == 0 || rows[index - 1].upNext != row.upNext) { if (index == 0 || rows[index - 1].upNext != row.upNext) {
// Keyed by index, not section: the context sits on both sides of the
// block, so a section-based key would duplicate.
item(key = "header-$index") { item(key = "header-$index") {
QueueSectionHeader( QueueSectionHeader(
label = if (row.upNext) "Up next" else contextTitle ?: "Next up", label = if (row.upNext) "Up next" else contextTitle ?: "Next up",