Remove compose ui fallback from LocalViewModelStoreOwner about compose-multiplatform HOT 3 CLOSED

Thanks for getting in touch. Let me summarize my thoughts from our DM here.

suggestion would be remove the fallback on LocalViewModelStoreOwner for KMP targets (jbMain)

I believe this should be the same across the platforms

This is where it depends on Compose UI

Yes, but Android depends on UI only in similar line too.

I don't see why it can't have dependency in viewmodel-compose

This module will bring dependency not only to itself - it has dependencies that UI doesn't. We'd like to avoid extra dependencies in Compose UI module.

My suggestion here is to open an issue in Google's tracker and try to do such a revert in AOSP first.

I already made this change for compose-ui and lifecycle-compose for the same reason, so I suspect a similar change could be made here.

Yep, thanks for that. But I guess there the situation is a bit different - it really depends on UI part in terms of finding UI view as default provider. My best guess here at the moment is to extract the required part rom ui to separate ui-something module and depend only on that.

Anyway, I'm going to close it as not planned for now, since I believe that it should be done in AOSP first. Please post a link to the issue in AOSP once created.

from compose-multiplatform.

@DevSrSouza, could you please open an issue in our AOSP lifecycle issue tracker so we can investigate this further? Please cross-link both issues for additional context and easier tracking. Thank you!

from compose-multiplatform.

The dependency between compose-ui and viewmodel-compose should likely be inverted. I already made this change for compose-ui and lifecycle-compose for the same reason, so I suspect a similar change could be made here.

from compose-multiplatform.