As ever a great video, I won't be implementing this it feels like you've just created a new problem, but as ever you are a great dev if I did want to do it this feels like a clean way to go about it
Thanks for the video. It seems more clear when navigation is done from ViewModel (as navigation is also a part of screen business logic). I also end up using approach very close to this one for the navigation in my open-source android app.
We have similar "custom" navigation but recetly migrated to "vanilla" navigation because it didn't play well with deeplinks. Reason was that Compose deeplink navigation didn't use navigation flow from custom nav so whole navigation got into inconsistent state. Plus benefit of navigation by pure callbacks from Compose component is that lifecycle is handled by default.
Furthermore what I like to do with this approach is using delegation and implementing the Navigator interface in the view models. That way (especially for simpler navigations) you don’t have to create the functions in each viewModel
The viewmodel shouldn't know anything about ui logic like navigation - just like the ui doesn't know what the viewmodel does. What's next? Are we going to start rendering composables in viewmodels too?
How is this any different from just sending events from Viewmodel and navigating in the Activity/Fragment/Composable ? The ideal setup is just create a navigator class which wraps the navcontroller and all navigation is done within Navigator class itself.
Muchas gracias, soy nueva en jetpack compose y estaba buscando orientacion de como hacer una ruta serializadle para pasarla a una BottonAppBard. Con este video aclaraste mi duda, tus videos son muy buenos, es una lastima que no haya contenido como el tuyo en español, asi que por el momento tambien doy gracias a YT por los subtitulos xD
Hello. I have updated Android Studio to the latest version and now the default parameters are not displayed when creating any component. What do I have to activate to show them again? Your videos are very good! Thank you!
The biggest problem with TDD in Android development and why there's not many people using it, is because it's a process that implies a very fast compilation time. It's not productive to have to wait 1-2 minutes or even more to do a small change. I love the concept though!
the most perfect comment. This video is definitely highly controversial. 😅😅 Well, honestly, as far as my opinion goes, I'm on the traditional train & I don't necessarily see the need to jump on this modern train. 😀😀 What's your take on this?
The current approach. Let's say a button clicks, then it calls a function in the VM, then the VM emits a navigation event back to the Composable, then the Composable invokes a callback to the parent Composable. All the way up to the NavGraph declaration where some class instance will parse this event and determine where to go next. I think this is a lot of boilerplate, why not having the class instance that decide the navigation in the VM itself, properly injected. And when the button click event comes, just determine where to go there. No ceremony. I actually like this approach
I'm not sure this is a good idea, but I will let you cook, what happens when the view is disposed and the view model isn't and it tries to navigate you somewhere that can't exist
@@PhilippLackner yh I might be talking about something that would never happen, essentially you mention the opposite when you said a shared flow would lose emissions while in the background, I'm saying what happens when it starts replaying them and the app is in a different place to where it should be, but I simply mean this could unintentionally navigate a user somewhere without them touching anything I feel navigation should only be handled by the UI
@@martinseal1987 I think what he is proposing here was a solution he invented for a usecase with a client or company he worked with and it seemed like what was proposed by default by the framework was not satisfactory so he moved with that. I can see that the whole point of that is to be testable. Basically you test the navigation from testing the viewmodel and swapping the implementation in the UI test because Navigator is an interface. Let me remember.... before compose. yeah we could test it .. OnView blah blah we were doing a text recon to see if after a performClick it was there with compose we are doing the same thing. I think here they are doing that VM driven navigation to centralise the inteligence of the navigation into each part of the app. Meh before we were using state change and the view reacted to that to change what it was displaying and to test we were simply verifying the change of state that should match the usecase... Well I do not see WHY we would do that. clearly maybe it's useless ( but I am surely mistaken just because I do not see something doesn't mean it is not there )
what if we need to pass navOptions to navigate action, or we have a bottom nav bar and we need to save 2 nav controllers for that? Then we will save 2 navigators with separate implementation for bottom nav bar and other screens. I think navigator class just a boilerplate and not clean way for navigation handling.
it's just an abstraction helping you to test the code. you can write all the stuff in single activity, but don't expect to be hired by decent company :)
I don't see how it will work in case of multimodule project where every feature is a module with own navigation and destinations. The inheritance of sealed interface will not work between modules.
Navigation is not the responsibility of a feature module. The app module has to wire everything together either way, or you destroy the reusability of your modules.
I will add to this, if you are familiar with iOS, the "Coordinator pattern" aims to centralize navigation logic and make individual modules or view controllers (with their respective coordinators) reusable and independent. In Android, using the app module to coordinate navigation serves the same purpose as a Coordinator in iOS: it decouples navigation from the individual feature modules (or view controllers in iOS), ensuring that transitions between screens are managed centrally.
Where is it coupled to Compose 🤨 In XML you'd just use your XML navigator to navigate when listening to the navigation events and put that in the activity.
@@PhilippLackner This means that a higher abstraction needs to be used in order to use this approach logically, otherwise it just complicates the work.
25 minutes of video at high speed just to navigate to another screen, really? Koin a DI, really? Koin is a service locator. Just saying, in today world android is not android anymore, compose ruined it..... easy to buid UI but good luck fixing performance...
As ever a great video, I won't be implementing this it feels like you've just created a new problem, but as ever you are a great dev if I did want to do it this feels like a clean way to go about it
We are already using this navigator but navigation is generally done from activities through events
And that's the proper way.
Thanks for the video. It seems more clear when navigation is done from ViewModel (as navigation is also a part of screen business logic). I also end up using approach very close to this one for the navigation in my open-source android app.
My compose app is recomposing. If a viewstate in mainactivity is changing, it recomposes full navigation as well. This is pain
We have similar "custom" navigation but recetly migrated to "vanilla" navigation because it didn't play well with deeplinks. Reason was that Compose deeplink navigation didn't use navigation flow from custom nav so whole navigation got into inconsistent state. Plus benefit of navigation by pure callbacks from Compose component is that lifecycle is handled by default.
Furthermore what I like to do with this approach is using delegation and implementing the Navigator interface in the view models. That way (especially for simpler navigations) you don’t have to create the functions in each viewModel
Whenever I've ever tried to wrap a navigator it has ended in tears
Thanks Philip, your videos are golden
But why handle the navigation (which is ui logic) inside a viewmodel?
Why not?
@@xybnedasdd2930 it violates the separation of concerns principle
@@volod-one please tell him.
The viewmodel shouldn't know anything about ui logic like navigation - just like the ui doesn't know what the viewmodel does. What's next? Are we going to start rendering composables in viewmodels too?
@@denisgithuku8563 Are we going to start rendering composables in viewmodels too?
😂
How is this any different from just sending events from Viewmodel and navigating in the Activity/Fragment/Composable ? The ideal setup is just create a navigator class which wraps the navcontroller and all navigation is done within Navigator class itself.
Muchas gracias, soy nueva en jetpack compose y estaba buscando orientacion de como hacer una ruta serializadle para pasarla a una BottonAppBard. Con este video aclaraste mi duda, tus videos son muy buenos, es una lastima que no haya contenido como el tuyo en español, asi que por el momento tambien doy gracias a YT por los subtitulos xD
U are keeping navigation logic in Custom Nav Host class and now are u are keeping in viewmodel which is preferrable?
Hello.
I have updated Android Studio to the latest version and now the default parameters are not displayed when creating any component.
What do I have to activate to show them again?
Your videos are very good!
Thank you!
Instead of a Singleton you can scope one instance per NavGraph
how is that possible?
if you take a bit of inspiration from Slack's Circuit, you can make this even cleaner
do you have a link for this? im really intrigued on this approach
@@peaceka you'll see that it's not just about navigation but navigation is a part of it
@@markonovakovic3838 aight, thanks. found it.
@@markonovakovic3838 using compositionLocal right?
Will you do a video on TDD? I've been looking to apply it, but don't really have a solid workflow for UI tests
The biggest problem with TDD in Android development and why there's not many people using it, is because it's a process that implies a very fast compilation time. It's not productive to have to wait 1-2 minutes or even more to do a small change.
I love the concept though!
How about single responsibility rule and that viewmodel?
This one is going to be controversial 😅
the most perfect comment. This video is definitely highly controversial. 😅😅
Well, honestly, as far as my opinion goes, I'm on the traditional train & I don't necessarily see the need to jump on this modern train. 😀😀
What's your take on this?
Gotcha 👍
The current approach. Let's say a button clicks, then it calls a function in the VM, then the VM emits a navigation event back to the Composable, then the Composable invokes a callback to the parent Composable. All the way up to the NavGraph declaration where some class instance will parse this event and determine where to go next.
I think this is a lot of boilerplate, why not having the class instance that decide the navigation in the VM itself, properly injected. And when the button click event comes, just determine where to go there. No ceremony.
I actually like this approach
I love Android evolution: just in 20min+ of ⚡ fast coding + set of supporting libraries you could create navigation between 3 screens 😂😢😂
How would you adjust this ObserveAsEvents function to work with KMP while implementing the repeatOnLifecycle part?
Who cares about KMP, its all hype
@@aabhishek4911 what is the benefit for trashing kmp? it would be incredible for us, android developers, if kmp succeed
implement it via expect/actual for all platforms
@@aabhishek4911 is it though? Provide facts please, then I'll listen and maybe believe it.
Thanks for video Philip. As far as I am concerned we can use the same way to show snackbar message. Isn't it?
Hi.....How to show a floating window over applications in Jetpack Compose Foreground
How would I go about adding Bottom Navigation in the Home Graph?
What are the performance implications of this? Would navigation still be snappy in large applications?
If i use compose multiplatform and use voyager for navigation how navigation from any methods like when received data from firebase messaging
Hello. guys! Any idea on how to implement this with nested navigation ????
I'm not sure this is a good idea, but I will let you cook, what happens when the view is disposed and the view model isn't and it tries to navigate you somewhere that can't exist
@@martinseal1987 I'm not sure I'm getting your point with navigating somewhere that doesn't exist.
@@PhilippLackner yh I might be talking about something that would never happen, essentially you mention the opposite when you said a shared flow would lose emissions while in the background, I'm saying what happens when it starts replaying them and the app is in a different place to where it should be, but I simply mean this could unintentionally navigate a user somewhere without them touching anything I feel navigation should only be handled by the UI
@@martinseal1987 I think what he is proposing here was a solution he invented for a usecase with a client or company he worked with and it seemed like what was proposed by default by the framework was not satisfactory so he moved with that.
I can see that the whole point of that is to be testable.
Basically you test the navigation from testing the viewmodel and swapping the implementation in the UI test because Navigator is an interface.
Let me remember.... before compose. yeah we could test it .. OnView blah blah we were doing a text recon to see if after a performClick it was there with compose we are doing the same thing.
I think here they are doing that VM driven navigation to centralise the inteligence of the navigation into each part of the app.
Meh before we were using state change and the view reacted to that to change what it was displaying and to test we were simply verifying the change of state that should match the usecase...
Well I do not see WHY we would do that.
clearly maybe it's useless ( but I am surely mistaken just because I do not see something doesn't mean it is not there )
Yeah, I also think that navigating should be handled by ui.
@@dvg-in-eu It still handled by ui. This is just an abstraction over navgation. Makes tesing easy.
I send HOF from compose to the viewmodel
what if we need to pass navOptions to navigate action, or we have a bottom nav bar and we need to save 2 nav controllers for that?
Then we will save 2 navigators with separate implementation for bottom nav bar and other screens.
I think navigator class just a boilerplate and not clean way for navigation handling.
it's just an abstraction helping you to test the code. you can write all the stuff in single activity, but don't expect to be hired by decent company :)
I don't see how it will work in case of multimodule project where every feature is a module with own navigation and destinations. The inheritance of sealed interface will not work between modules.
Navigation is not the responsibility of a feature module. The app module has to wire everything together either way, or you destroy the reusability of your modules.
I will add to this, if you are familiar with iOS, the "Coordinator pattern" aims to centralize navigation logic and make individual modules or view controllers (with their respective coordinators) reusable and independent. In Android, using the app module to coordinate navigation serves the same purpose as a Coordinator in iOS: it decouples navigation from the individual feature modules (or view controllers in iOS), ensuring that transitions between screens are managed centrally.
Your solution is coupled with Compose Navigation, if we decide to replace it with another navigation library the whole code must be dumped.
Where is it coupled to Compose 🤨
In XML you'd just use your XML navigator to navigate when listening to the navigation events and put that in the activity.
@@PhilippLackner Not Compose, Navigation Library
Then create your own NavOptions object and it's uncoupled 🤷🏼♂️ seemed overkill for the video.
@@PhilippLackner This means that a higher abstraction needs to be used in order to use this approach logically, otherwise it just complicates the work.
First here
25 minutes of video at high speed just to navigate to another screen, really? Koin a DI, really? Koin is a service locator. Just saying, in today world android is not android anymore, compose ruined it..... easy to buid UI but good luck fixing performance...
I do not recommend this.