Fix back navigation issue, when opening directly the notification troubleshoot screen.
This commit is contained in:
parent
2e8b63c006
commit
059c0dfe59
2 changed files with 31 additions and 1 deletions
|
|
@ -44,6 +44,7 @@ import io.element.android.features.preferences.impl.root.PreferencesRootNode
|
||||||
import io.element.android.features.preferences.impl.user.editprofile.EditUserProfileNode
|
import io.element.android.features.preferences.impl.user.editprofile.EditUserProfileNode
|
||||||
import io.element.android.libraries.architecture.BackstackView
|
import io.element.android.libraries.architecture.BackstackView
|
||||||
import io.element.android.libraries.architecture.BaseFlowNode
|
import io.element.android.libraries.architecture.BaseFlowNode
|
||||||
|
import io.element.android.libraries.architecture.appyx.canPop
|
||||||
import io.element.android.libraries.architecture.createNode
|
import io.element.android.libraries.architecture.createNode
|
||||||
import io.element.android.libraries.di.SessionScope
|
import io.element.android.libraries.di.SessionScope
|
||||||
import io.element.android.libraries.matrix.api.core.RoomId
|
import io.element.android.libraries.matrix.api.core.RoomId
|
||||||
|
|
@ -190,7 +191,11 @@ class PreferencesFlowNode @AssistedInject constructor(
|
||||||
notificationTroubleShootEntryPoint.nodeBuilder(this, buildContext)
|
notificationTroubleShootEntryPoint.nodeBuilder(this, buildContext)
|
||||||
.callback(object : NotificationTroubleShootEntryPoint.Callback {
|
.callback(object : NotificationTroubleShootEntryPoint.Callback {
|
||||||
override fun onDone() {
|
override fun onDone() {
|
||||||
backstack.pop()
|
if (backstack.canPop()) {
|
||||||
|
backstack.pop()
|
||||||
|
} else {
|
||||||
|
navigateUp()
|
||||||
|
}
|
||||||
}
|
}
|
||||||
})
|
})
|
||||||
.build()
|
.build()
|
||||||
|
|
|
||||||
|
|
@ -0,0 +1,25 @@
|
||||||
|
/*
|
||||||
|
* Copyright (c) 2024 New Vector Ltd
|
||||||
|
*
|
||||||
|
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||||
|
* you may not use this file except in compliance with the License.
|
||||||
|
* You may obtain a copy of the License at
|
||||||
|
*
|
||||||
|
* http://www.apache.org/licenses/LICENSE-2.0
|
||||||
|
*
|
||||||
|
* Unless required by applicable law or agreed to in writing, software
|
||||||
|
* distributed under the License is distributed on an "AS IS" BASIS,
|
||||||
|
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||||
|
* See the License for the specific language governing permissions and
|
||||||
|
* limitations under the License.
|
||||||
|
*/
|
||||||
|
|
||||||
|
package io.element.android.libraries.architecture.appyx
|
||||||
|
|
||||||
|
import com.bumble.appyx.navmodel.backstack.BackStack
|
||||||
|
|
||||||
|
fun <T : Any> BackStack<T>.canPop(): Boolean {
|
||||||
|
val elements = elements.value
|
||||||
|
return elements.any { it.targetState == BackStack.State.ACTIVE } &&
|
||||||
|
elements.any { it.targetState == BackStack.State.STASHED }
|
||||||
|
}
|
||||||
Loading…
Add table
Add a link
Reference in a new issue