[FR]: Use navigateUp() Instead of popBackStack() for Toolbar Up Button Click

[imjoz]

Is there an existing issue for this?

• I have searched the existing issues

Describe the problem

I recently reviewed the principles of navigation in Compose. From what I understand, the top app bar’s “Up” navigation button should call navController.navigateUp() rather than popBackStack(), which is intended for handling the device’s back press.

However, in the NiaNavHost, the onBackClick callbacks in various screens are currently using popBackStack(). My understanding is that onBackClick is hoisted to handle the toolbar’s “Up” button clicks in these screens.

The current implementation using popBackStack() works as expected. However, I would like to understand whether using navigateUp() might be a more appropriate approach.

Describe the solution

Current

topicScreen(
    showBackButton = true,
    onBackClick = navController::popBackStack,
    onTopicClick = navController::navigateToTopic,
)

Proposed

topicScreen(
    showBackButton = true,
    onNavigateUp = navController::navigateUp,
    onTopicClick = navController::navigateToTopic,
)

Additional context

No response

Code of Conduct

• I agree to follow this project's Code of Conduct

[akash0228]

if anyone confirms i can work on it

[imjoz]

Thanks @akash0228, I already have a PR line-up after I created the issue, I was also waiting for the confirmation, cheers ✨