search

JetBrains / compose-multiplatform

Compose Multiplatform, a modern UI framework for Kotlin that makes building performant and beautiful user interfaces easy and enjoyable.
https://jetbrains.com/lp/compose-multiplatform
Apache License 2.0
15.17k stars 1.11k forks source link

Fix ImageViewer: Apply applyDefaultHierarchyTemplate #4762

Closed eymar closed 1 month ago

eymar commented 1 month ago

The intermediate iosMain source was previously removed here: https://github.com/JetBrains/compose-multiplatform/pull/3956 And it worked fine.

According to the doc https://kotlinlang.org/docs/whatsnew1920.html#set-up-the-target-hierarchy :

If you want to have additional source sets that the default hierarchy template doesn't provide, for example, one that shares code between a macOS and a JVM target, adjust the hierarchy by reapplying the template explicitly with applyDefaultHierarchyTemplate() and configuring additional source sets manually as usual with dependsOn()

Now we have an additional custom source set named 'jsWasmMain', therefore we need to have applyDefaultHierarchyTemplate().

eymar commented 1 month ago

A trivial change. So merging it right away.