Javier
09/27/2021, 1:28 PMapi
to implementation
?Javier
09/27/2021, 1:28 PM...
commonMain.dependencies {
implementation(libs.kotlinStdlibCommon)
implementation(project(":compose:ui:ui-util"))
api(project(":compose:foundation:foundation"))
api(project(":compose:material:material"))
api(project(":compose:runtime:runtime"))
api(project(":compose:ui:ui"))
api(project(":compose:ui:ui-tooling-preview"))
}
...
Javier
09/27/2021, 1:32 PMIgor Demin
09/27/2021, 1:37 PMapi
with implementation
)
The purpose of desktop
was aggregating all Compose modules, and it was introduced at the very beginning of the CfD. But as you mention, not everyone needs material, also not everyone need foundation.
The current workaround is to exclude dependency explicitly.Igor Demin
09/27/2021, 1:38 PMit is exposing this theme, which, IMO, should be in its own artifact like material does in AndroidDesktopTheme will be deprecated in a new build, so this module will no longer have usable classes/functions
Javier
09/27/2021, 1:40 PMJavier
09/27/2021, 1:40 PMJavier
09/27/2021, 1:40 PMIgor Demin
09/27/2021, 1:41 PMMaterialTheme
, DesktopTheme by CompositionLocalProvider(LocalScrollbarStyle provides...
Igor Demin
09/27/2021, 1:46 PMthen I can just wait until this module is deletedIt maybe not in the near future, because it is not trivial to get rid of it currently: •
desktop
knows what version of skiko should be used (when we use desktop.currentOs
we actually use desktop-windows
or desktop-linux
).
• we want to keep one-liner dependency for users, which use only material
module
the issue suggests how we can fix it.Javier
09/27/2021, 1:47 PMTheoretically we can still ship separate artifacts for each platform (mac-arm64, mac-x64, win64, linux64), and make MPP plugin resolve platform automatically: