Sulkta-OSS/element-x-ada
3
0
Fork 0
Code Issues 1 Pull requests 10 Projects Releases Packages Wiki Activity Actions

Make sure we don't finish NotificationTapOpensTimeline until we've actually rendered the timeline event we need to focus on

Browse source
This commit is contained in:
Jorge Martín 2025-11-28 08:25:26 +01:00 • committed by Jorge Martin Espinosa
parent 568c1ae90d
commit 7a1dd24dbd
1 changed files with 3 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/TimelinePresenter.kt
View file

@ -206,6 +206,9 @@ class TimelinePresenter(
focusOnEvent(event.eventId, focusRequestState) focusOnEvent(event.eventId, focusRequestState)
}.start() }.start()
is TimelineEvents.OnFocusEventRender -> { is TimelineEvents.OnFocusEventRender -> {
// If there was a pending 'notification tap opens timeline' transaction, finish it now we're focused in the required event
analyticsService.finishLongRunningTransaction(NotificationTapOpensTimeline)
focusRequestState.value = focusRequestState.value.onFocusEventRender() focusRequestState.value = focusRequestState.value.onFocusEventRender()
} }
is TimelineEvents.ClearFocusRequestState -> { is TimelineEvents.ClearFocusRequestState -> {
@ -242,7 +245,6 @@ class TimelinePresenter(
analyticsService.run { analyticsService.run {
finishLongRunningTransaction(DisplayFirstTimelineItems) finishLongRunningTransaction(DisplayFirstTimelineItems)
finishLongRunningTransaction(OpenRoom) finishLongRunningTransaction(OpenRoom)
finishLongRunningTransaction(NotificationTapOpensTimeline)
} }
} }
.launchIn(this) .launchIn(this)