fix: address regression in comment collapse

This commit is contained in:
Harsh Shandilya 2025-06-02 21:52:19 +05:30
parent 8947c3a3bc
commit 764be07cc0
2 changed files with 4 additions and 12 deletions

View file

@ -10,9 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### Changed ### Changed
- The navigation stack has been completely rewritten to better - The navigation stack has been completely rewritten to better
support larger screens. This also adds some nicer animations support larger screens
and better support for predictive back. There will continue
to be iterative improvements in this area.
- Comment expand/collapse interaction is now driven by a - Comment expand/collapse interaction is now driven by a
button UI to prevent misclicks button UI to prevent misclicks
@ -22,6 +20,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Upgrade to Compose 2025-05 releases - Upgrade to Compose 2025-05 releases
- Increase swipe action trigger distance to compensate for - Increase swipe action trigger distance to compensate for
gesture sensitivity changes in Android 16 QPR1 Beta 1 gesture sensitivity changes in Android 16 QPR1 Beta 1
- Fix regression in collapsing deeply nested comment trees
## [1.55.0] - 2025-02-13 ## [1.55.0] - 2025-02-13

View file

@ -62,14 +62,7 @@ internal class CommentsHandler {
return node.copy(children = updatedChildren) return node.copy(children = updatedChildren)
} }
val listNode = _listItems.value.toMutableList() val listNode = _listItems.value.map { updateNode(it) }
for (i in listNode.indices) { _listItems.value = listNode
val node = listNode[i]
if (node.comment.shortId == shortId || node.children.any { it.comment.shortId == shortId }) {
listNode[i] = updateNode(node)
_listItems.value = listNode.toList()
return
}
}
} }
} }