Oleksandr Karpovich
Oleksandr Karpovich
Reproduced: ```kotlin @Composable @Preview fun App() { var currentText: String by remember { mutableStateOf("") } Column { BasicText( text = currentText, modifier = Modifier.width(100.dp), onTextLayout = { result -> val...
The reproducer: https://github.com/Kotlin/kmm-production-sample/tree/issue-2929 `./gradlew :composeApp:jsBrowserRun -i`
Related thread in Slack https://kotlinlang.slack.com/archives/C01D6HTPATV/p1679036069089779
Reproduced on MacOs too: https://github.com/eymar/repr_compose_2891 Compose: 1.3.1 Kotlin: 1.8.10 I confirm that it works in compose 1.1.1
@starglad Thanks! The issue is clear and reproducible. What behaviour did you want to achieve my typing while keeping the mouse button pressed after selection + moving the mouse afterwards?...
Nice! Yes, create a PR please
Please link the PR here when you have it ready.
No problem at all. Take your time. Just wanted to be sure not to miss the PR.
https://android-review.googlesource.com/c/platform/frameworks/support/+/2028663/27/compose/ui/ui-text/src/androidMain/kotlin/androidx/compose/ui/text/AndroidTextStyle.android.kt#116 - `includeFontPadding` was added on Android for compatibility reasons and it will be removed. Have you encountered a use case where `includeFontPadding` would help on desktop? (A use case...
Thank you @litrik. We usually aim to align compose-jb with Jetpack Compose behaviour. So we'll look at when we can implement this.