CompositionLocals given in application scope are not take into account in window scope (such as LocalLayoutDirection)

[mahozad]

Describe the bug
All Arrangement.Absolute.spacedBy() overloads refer to Arrangement.spacedBy() in their documentation for the direction-aware alternative.

But the Arrangement.spacedBy() does not honor RTL layout direction.

Affected platforms

• Desktop
• and probably others

Versions

• Kotlin version*: 1.9.0
• Compose Multiplatform version*: 1.5.0-rc01
• OS version(s)*: Windows 10, Windows 11
• OS architecture (x86 or arm64): x64
• JDK (for desktop issues): Oracle JDK 17.0.7

To Reproduce

// ...
import androidx.compose.foundation.layout.Arrangement.spacedBy

fun main() {
    application {
        CompositionLocalProvider(LocalLayoutDirection provides LayoutDirection.Rtl) {
            Window(onCloseRequest = ::exitApplication) {
                Row(horizontalArrangement = spacedBy(24.dp)) {
                    Box(modifier = Modifier.size(64.dp).background(Color.Red))
                    Box(modifier = Modifier.size(64.dp).background(Color.Green))
                }
            }
        }
    }
}

Expected behavior
When layout direction is RTL, the components should be placed in reverse order (green box should be on the left).

[mahozad]

When I moved CompositionLocalProvider into Window content, it works correctly:

fun main() {
    application {
        Window(onCloseRequest = ::exitApplication) {
            CompositionLocalProvider(LocalLayoutDirection provides LayoutDirection.Rtl) {
                Row(horizontalArrangement = spacedBy(24.dp)) {
                    Box(modifier = Modifier.size(64.dp).background(Color.Red))
                    Box(modifier = Modifier.size(64.dp).background(Color.Green))
                }
            }
        }
    }
}

So, it should be the same problem as this:

• LocalLayoutDirection isn't propagated into DialogWindow #3382

[Walingar]

Hello @mahozad, thank you for submitting the issue.
This is related to this issue: #3289. See the comment

For now we provide CompositionLocals in Window scope and given LayoutDirection is overriden by default values

[mahozad]

Thanks for the relevant information.

[m-sasha]

Fixed by JetBrains/compose-multiplatform-core#777