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

since these are in public API (:/) we need to do this in 1.2

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

since it is already marked as deprecated, we can probably do it by now.

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

Opening diff shortly

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

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

commit d576cbdc911cba16638a44fd8223391a90a07ef7
Author: Mike Nakhimovich <digitalbuddha@users.noreply.github.com>
Date: Tue Aug 11 09:30:34 2020

[GH] Hide deprecated internal API.

## Proposed Changes

* `RoomDatabase.java` has protected `mCallbacks` field which is leaking in the API docs, we should @Hide it.

## Testing

Test: Ran unit tests locally

## Issues Fixed

Fixes: 76109329

This is an imported pull request from

https://github.com/androidx/androidx/pull/61.

Resolves #61
Github-Pr-Head-Sha: 6440daa3a63752c7f9d5ba2a390248cd85bc634f
GitOrigin-RevId: fe92d8466a59b44b218b6ca3cbd57dcda17992f7

Change-Id: Id599cdf5b02b32bdae0166266fb7da967598fe92

A room/runtime/api/current.ignore
M room/runtime/api/current.txt
M room/runtime/api/public_plus_experimental_current.txt
M room/runtime/api/restricted_current.txt
M room/runtime/src/main/java/androidx/room/RoomDatabase.java

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

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