Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

For #305 - Add support for Compose multiplatform #306

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

Mugurell
Copy link

Add support for Compose Multiplatform - https://blog.jetbrains.com/kotlin/2023/05/compose-multiplatform-for-ios-is-in-alpha/

More details available in the individual commit's message.
The gist of changes is that all the UI is now moved from :app to :shared and with the help of Compose Multiplatform both Android and iOS apps will share not just the same functionality but also the same UI.

For maintainers: I think this can be a separate branch in the project. I can ask merging in a new more suitable branch if one is created for this.

Android result iOS result
KaMPKit.-.Compose.Multiplatform.on.Android.mp4
kampkit-compose-multiplatform-on-ios_WMRq8CUN.mp4

Mugurell added 6 commits July 21, 2023 19:32
…OS app

This will ensure the iOS app can be built.
This will allow to share it with iOS also.
Migrated to Material3 and Dynamic Color support for Android in the meantime.
Migrated to support automatic day/night theme support for both platforms.
…red on Android

Added moko-resources as the community preferred approach to using strings in
similar scenarios.
Added the materialIconsExtended dependency for :shared instead of reusing the
vector images shown previously from :app since those vectors are not in a
format moko-resources supports at this moment (though it support SVGs). While
this is a big package R8 will ensure unused resources are stripped from the
generated builds at compile time.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant