Fixed

Bug

Status Update

No update yet.

Description

[Deleted User]

created issue #1

Apr 21, 2023 06:57PM

Versions

Component used: Navigation

Version used: 2.5.3

Devices/Android versions reproduced on: any

Description

According to the documentation of [BackHandler], nested BackHandlers(https://developer.android.com/reference/kotlin/androidx/activity/compose/package-summary#top-level-functions) should take priority:

If this is called by nested composables, if enabled, the inner most composable will consume the call to system back and invoke its lambda. The call will continue to propagate up until it finds an enabled BackHandler.

However, this doesn't work correctly if the activity is paused and then resumed.

Steps to reproduce

Start the app
Click back button
Toast "Button clicked" shows
Minimize the app (by pressing home button)
Bring app to foreground (via recent apps menu)
Click back button Expected behavior: Toast "Button clicked" shows Actual behavior: Toast "This should never ever show!!!" shows

Code to demonstrate the issue

class MainActivity : ComponentActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContent {
            MaterialTheme {
                Surface(Modifier.fillMaxSize()) {
                    val context = LocalContext.current
                    BackHandler {
                        Toast.makeText(context, "This should never ever show!!!", Toast.LENGTH_SHORT).show()
                    }
                    SampleApp()
                }
            }
        }
    }
}

@Composable
fun SampleApp() {
    val navController = rememberNavController()
    NavHost(navController, "first") {
        composable("first") { Screen() }
    }
}

@Composable
fun Screen() {
    val context = LocalContext.current
    BackHandler {
        Toast.makeText(context, "Back clicked", Toast.LENGTH_SHORT).show()
    }
    Column {
        Text("First screen")
    }
}

A sample project is attached.

MyApplication.zip

29 MB

Download

Comments

jb...@google.com <jb...@google.com> #2Apr 25, 2023 11:08PM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit c60f33e229e31ab328ef6b59dab63b264954831c
Author: Alexandre Elias <aelias@google.com>
Date: Fri Jul 10 16:23:09 2020

Semantics no-op cleanups

Partly in response to lmr's broad code review, I did a pass of
superficial API/implementation cleanup. The main changes are:

- I changed each Boolean SemanticsProperty where false is equivalent to
not being present to take "Unit" instead. This is conceptually
clearer: it avoids questions like "can I cancel out a semantics from a
merged child by setting it to false?" Because "property = Unit" looks
weird, I also changed the style of these to "property()".

- I moved the Semantics id generator closer to where it's used, in
SemanticsModifierCore. I made it internal and an AtomicInt.
(Note that integer ids are heavily used in the Android
AccessibilityNodeInfo APIs so I can't simply remove them entirely.)

- I deleted dead code. Some examples include SemanticsHintOverrides,
a public API not connected to anything, and SemanticsPropertyKey
merge() open method which is never called. (In both cases I have
a different plan in mind for accessibility.)

Fixes: 145951226
Fixes: 145955412
Test: existing tests
Relnote: "Single-value semantics properties now use a calling style.
For example, 'semantics { hidden = true }' is now written as:
'semantics { hidden() }'."

Change-Id: Ic1afd12ea22c926babc9662f1804d80b33aa0cfc

M ui/integration-tests/benchmark/src/androidTest/java/androidx/ui/benchmark/test/LayoutNodeModifierBenchmark.kt
M ui/ui-core/api/0.1.0-dev15.txt
M ui/ui-core/api/current.txt
M ui/ui-core/api/public_plus_experimental_0.1.0-dev15.txt
M ui/ui-core/api/public_plus_experimental_current.txt
M ui/ui-core/api/restricted_0.1.0-dev15.txt
M ui/ui-core/api/restricted_current.txt
M ui/ui-core/src/androidAndroidTest/kotlin/androidx/ui/graphics/vector/VectorTest.kt
M ui/ui-core/src/androidAndroidTest/kotlin/androidx/ui/semantics/SemanticsTests.kt
M ui/ui-core/src/androidMain/kotlin/androidx/ui/core/AndroidActuals.kt
M ui/ui-core/src/androidMain/kotlin/androidx/ui/core/AndroidComposeView.kt
M ui/ui-core/src/androidMain/kotlin/androidx/ui/core/AndroidComposeViewAccessibilityDelegateCompat.kt
M ui/ui-core/src/androidMain/kotlin/androidx/ui/core/AndroidPopup.kt
M ui/ui-core/src/commonMain/kotlin/androidx/ui/core/Expect.kt
M ui/ui-core/src/commonMain/kotlin/androidx/ui/core/semantics/SemanticsConfiguration.kt
D ui/ui-core/src/commonMain/kotlin/androidx/ui/core/semantics/SemanticsHintOverrides.kt
M ui/ui-core/src/commonMain/kotlin/androidx/ui/core/semantics/SemanticsModifier.kt
M ui/ui-core/src/commonMain/kotlin/androidx/ui/core/semantics/SemanticsNode.kt
M ui/ui-core/src/commonMain/kotlin/androidx/ui/core/semantics/SemanticsOwner.kt
M ui/ui-core/src/commonMain/kotlin/androidx/ui/core/semantics/SemanticsWrapper.kt
M ui/ui-core/src/commonMain/kotlin/androidx/ui/semantics/SemanticsProperties.kt
M ui/ui-foundation/api/0.1.0-dev15.txt
M ui/ui-foundation/api/current.txt
M ui/ui-foundation/api/public_plus_experimental_0.1.0-dev15.txt
M ui/ui-foundation/api/public_plus_experimental_current.txt
M ui/ui-foundation/api/restricted_0.1.0-dev15.txt
M ui/ui-foundation/api/restricted_current.txt
M ui/ui-foundation/src/main/java/androidx/ui/foundation/Clickable.kt
M ui/ui-foundation/src/main/java/androidx/ui/foundation/Dialog.kt
M ui/ui-foundation/src/main/java/androidx/ui/foundation/Scroller.kt
M ui/ui-foundation/src/main/java/androidx/ui/foundation/selection/Selectable.kt
M ui/ui-foundation/src/main/java/androidx/ui/foundation/selection/Toggleable.kt
M ui/ui-foundation/src/main/java/androidx/ui/foundation/semantics/FoundationSemanticsProperties.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/ButtonTest.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/CardTest.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/CheckboxScreenshotTest.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/RadioButtonScreenshotTest.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/ScaffoldTest.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/SnackbarTest.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/SurfaceTest.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/ripple/RippleIndicationTest.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/textfield/TextFieldScreenshotTest.kt
M ui/ui-material/src/main/java/androidx/ui/material/AppBar.kt
M ui/ui-material/src/main/java/androidx/ui/material/TextFieldImpl.kt
M ui/ui-test/src/androidTest/java/androidx/ui/test/AssertsTest.kt
M ui/ui-test/src/androidTest/java/androidx/ui/test/CallSemanticsActionTest.kt
M ui/ui-test/src/androidTest/java/androidx/ui/test/ErrorMessagesTest.kt
M ui/ui-test/src/androidTest/java/androidx/ui/test/FindersTest.kt
M ui/ui-test/src/androidTest/java/androidx/ui/test/PrintToStringTest.kt
M ui/ui-test/src/androidTest/java/androidx/ui/test/ScrollToTest.kt
M ui/ui-test/src/androidTest/java/androidx/ui/test/TextActionsTest.kt
M ui/ui-test/src/main/java/androidx/ui/test/Actions.kt
M ui/ui-test/src/main/java/androidx/ui/test/Filters.kt
M ui/ui-text/api/0.1.0-dev15.txt
M ui/ui-text/api/current.txt
M ui/ui-text/api/public_plus_experimental_0.1.0-dev15.txt
M ui/ui-text/api/public_plus_experimental_current.txt
M ui/ui-text/api/restricted_0.1.0-dev15.txt
M ui/ui-text/api/restricted_current.txt
M ui/ui-text/src/commonMain/kotlin/androidx/ui/text/CoreTextField.kt
M ui/ui-text/src/commonMain/kotlin/androidx/ui/text/TextSemanticsProperties.kt

https://android-review.googlesource.com/1360099

[Deleted User] <[Deleted User]> #3Apr 25, 2023 11:16PM

It only happens inside of a NavHost. So, yes, it seems to be specific to navigation.

il...@google.com <il...@google.com> Apr 26, 2023 11:44PM

Assigned to jb...@google.com.

jb...@google.com <jb...@google.com> #4Apr 27, 2023 12:26AM

This was caused by the fix to b/182284739 which made BackHandler lifecycle aware. The problem here is that the NavController registers its lifecycle and added an observer to the Activity Lifecycle in composition while the BackHandler registers its observer on the Activity Lifecycle in a DisposbleEffect. This means that the NavController will always get Lifecycle callbacks before the BackHandler, so the components listening the the NavController's lifecycle (like the NavBackStackEntry) will always get their Lifecycle callbacks before the BackHandler as well.

This should be addressed by work coming in 2.7 to integrate the BackHandler into the NavHost.

ap...@google.com <ap...@google.com> #5Jun 1, 2023 12:50AM

Project: platform/frameworks/support
Branch: androidx-main

commit d2c5efe67531a30fb9f1a129d52beb1e9ece0b29
Author: Jeremy Woods <jbwoods@google.com>
Date: Wed May 31 23:09:54 2023

Integrate BackHandler into NavHost

Instead of NavHost taking over the onBackPressedDispatcher from the
Activity, we should just make it use a BackHandler. This will ensure
that it interacters with the other BackHandlers in Compose correctly.

RelNote: "`NavHost` now correctly intercepts system back calls even
after the Activity has been `STOPPED` and `RESUMED`."
Test: modified tests
Bug: 279118447

Change-Id: Icb6deab996d122487243f0d3d775af8c15fc7c25

M navigation/navigation-compose/src/androidTest/java/androidx/navigation/compose/NavHostTest.kt
M navigation/navigation-compose/src/main/java/androidx/navigation/compose/NavHost.kt

https://android-review.googlesource.com/2610317

il...@google.com <il...@google.com> #6Jun 1, 2023 05:00PM

Marked as fixed.

This has been fixed internally and will be available in Navigation 2.7.0-beta01.

pr...@google.com <pr...@google.com> #7Jun 8, 2023 01:31AM

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.navigation:navigation-compose:2.7.0-beta01

an...@daresay.co <an...@daresay.co> #8Jun 26, 2023 12:35PM

Comment has been deleted.

Message last modified on Jun 26, 2023 12:35PM

li...@gmail.com <li...@gmail.com> #9Jun 26, 2023 12:38PM

I wouldn't say this is fixed. If you add a back-callback in the activity which is lifecycle aware, and then background->foreground the app, the activity back-callback will be put on top of the callback pile. This leads to that any BackHandler in your composable will NOT be triggered.

The best workaround I have for now is to use the non-lifecycle aware function in the activity.

class MainActivity : FragmentActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        
        // open fragment which hoists the composables
        ...

        if (savedInstanceState == null) {
             onBackPressedDispatcher.addCallback(onBackPressedCallback = someBackHandling())  
            // onBackPressedDispatcher.addCallback(owner = this, onBackPressedCallback = someBackHandling)  // not this one
        }
    }

Message last modified on Jun 26, 2023 12:40PM

ho...@gmail.com <ho...@gmail.com> #10Jul 17, 2023 01:51PM

Comment has been deleted.

Message last modified on Jul 17, 2023 03:43PM

ho...@gmail.com <ho...@gmail.com> #11Jul 17, 2023 02:31PM

Comment has been deleted.

Message last modified on Jul 17, 2023 03:43PM

ho...@gmail.com <ho...@gmail.com> #12Jul 18, 2023 12:10PM

https://developer.android.com/jetpack/androidx/releases/navigation#2.7.0-beta01 :

Bug Fixes : NavHost in Navigation Compose now correctly intercepts system back calls even after the Activity has been STOPPED and RESUMED.

It works fine for me.

Message last modified on Jul 18, 2023 03:45PM

gi...@gmail.com <gi...@gmail.com> #13Sep 22, 2023 12:43PM

I can confirm this is still happening as mentioned in <li...@gmail.com> #9

va...@zedge.net <va...@zedge.net> #14Oct 11, 2023 02:25PM

Still reproduces.
In my case it works fine when the app is first ran, but my custom callback calls navController.popBackStack internally if it can't handle back press on its own. So, when I run the app and click back, everything works, but after I pop up all the Composables from the stack and return to the app, the custom callback is not called anymore.

il...@google.com <il...@google.com> #15Oct 11, 2023 04:45PM

Re comment#14 - please file a new issue with a project that reproduces your problem.

Note that you should never, ever be calling navController.popBackStack() in your own BackHandler as Navigation must be the only one handling its own back stack with the system back button so that Predictive Back works correctly in the future.

ja...@gopuff.com <ja...@gopuff.com> #16Oct 30, 2023 07:13PM

These changes broke our use case to override back handling in navigation library for closing the menu drawer. Please see more details in the issue I've reported.

ja...@sense.com <ja...@sense.com> #17Nov 14, 2024 05:36PM

I believe this is still an issue even without using NavHost like mentioned in comment #16. I filed a new issue here

IssueTracker

BackHandler not working correctly after activity resumes after being paused

Status Update

Description

Versions

Description

Steps to reproduce

Code to demonstrate the issue

Comments

jb...@google.com <jb...@google.com> #2Apr 25, 2023 11:08PM

[Deleted User] <[Deleted User]> #3Apr 25, 2023 11:16PM

il...@google.com <il...@google.com> Apr 26, 2023 11:44PM

jb...@google.com <jb...@google.com> #4Apr 27, 2023 12:26AM

ap...@google.com <ap...@google.com> #5Jun 1, 2023 12:50AM

il...@google.com <il...@google.com> #6Jun 1, 2023 05:00PM

pr...@google.com <pr...@google.com> #7Jun 8, 2023 01:31AM

an...@daresay.co <an...@daresay.co> #8Jun 26, 2023 12:35PM

li...@gmail.com <li...@gmail.com> #9Jun 26, 2023 12:38PM

ho...@gmail.com <ho...@gmail.com> #10Jul 17, 2023 01:51PM

ho...@gmail.com <ho...@gmail.com> #11Jul 17, 2023 02:31PM

ho...@gmail.com <ho...@gmail.com> #12Jul 18, 2023 12:10PM

gi...@gmail.com <gi...@gmail.com> #13Sep 22, 2023 12:43PM

va...@zedge.net <va...@zedge.net> #14Oct 11, 2023 02:25PM

il...@google.com <il...@google.com> #15Oct 11, 2023 04:45PM

ja...@gopuff.com <ja...@gopuff.com> #16Oct 30, 2023 07:13PM

ja...@sense.com <ja...@sense.com> #17Nov 14, 2024 05:36PM

Issue 279118447

Description

Versions

Description

Steps to reproduce

Code to demonstrate the issue

Issue summary

Comments

jb...@google.com <jb...@google.com> #2Apr 25, 2023 11:08PM

[Deleted User] <[Deleted User]> #3Apr 25, 2023 11:16PM

il...@google.com <il...@google.com> Apr 26, 2023 11:44PM

jb...@google.com <jb...@google.com> #4Apr 27, 2023 12:26AM

ap...@google.com <ap...@google.com> #5Jun 1, 2023 12:50AM

il...@google.com <il...@google.com> #6Jun 1, 2023 05:00PM

pr...@google.com <pr...@google.com> #7Jun 8, 2023 01:31AM

an...@daresay.co <an...@daresay.co> #8Jun 26, 2023 12:35PM

li...@gmail.com <li...@gmail.com> #9Jun 26, 2023 12:38PM

ho...@gmail.com <ho...@gmail.com> #10Jul 17, 2023 01:51PM

ho...@gmail.com <ho...@gmail.com> #11Jul 17, 2023 02:31PM

ho...@gmail.com <ho...@gmail.com> #12Jul 18, 2023 12:10PM

gi...@gmail.com <gi...@gmail.com> #13Sep 22, 2023 12:43PM

va...@zedge.net <va...@zedge.net> #14Oct 11, 2023 02:25PM

il...@google.com <il...@google.com> #15Oct 11, 2023 04:45PM

ja...@gopuff.com <ja...@gopuff.com> #16Oct 30, 2023 07:13PM

ja...@sense.com <ja...@sense.com> #17Nov 14, 2024 05:36PM

Add comment

Issue metadata