Hi guys, I want to use `letterSpacing` in my `Text()` . I did it like this ```letterSpacing = 0.7.sp...

# compose

[KotlinLeaner]

Hi guys, I want to use letterSpacing in my Text() . I did it like this

letterSpacing = 0.7.sp

but when I move this value to dimen.xml and using like this

letterSpacing = dimensionResource(id = R.dimen.text_letter_spacing),

It giving me error on Text() . Error in thread

[KotlinLeaner]

None of the following functions can be called with the arguments supplied.
Text(AnnotatedString, Modifier = ..., Color = ..., TextUnit = ..., FontStyle? = ..., FontWeight? = ..., FontFamily? = ..., TextUnit = ..., TextDecoration? = ..., TextAlign? = ..., TextUnit = ..., TextOverflow = ..., Boolean = ..., Int = ..., Map<String, InlineTextContent> = ..., (TextLayoutResult) → Unit = ..., TextStyle = ...) defined in androidx.compose.material
Text(String, Modifier = ..., Color = ..., TextUnit = ..., FontStyle? = ..., FontWeight? = ..., FontFamily? = ..., TextUnit = ..., TextDecoration? = ..., TextAlign? = ..., TextUnit = ..., TextOverflow = ..., Boolean = ..., Int = ..., (TextLayoutResult) → Unit = ..., TextStyle = ...) defined in androidx.compose.material

[romainguy]

dimensionResource returns a value in dp but letterSpacing expects TextUnit

[KotlinLeaner]

So, is there no way to convert those values?

[Chrimaeon]

you can use the LocalDensity for convertion:

letterSpacing = with(LocalDensity.current){dimensionResource(id=R.dimen.text_letter_spacing).toSp()}

[KotlinLeaner]

It works. Thanks 🙏

[Chrimaeon]

Not sure though if it is the correct value. so dimensionResource will return a dp and with this method you are then converting this dp into sp

[romainguy]

No it’s fine

[romainguy]

sp are dp multiplied by the user selected font scale factor

[romainguy]

see https://developer.android.com/guide/topics/resources/more-resources#Dimension

[KotlinLeaner]

perfect 😊

[Colton Idle]

Just beware that letterSpacing doesn't play nice with elipssis text. Longstanding android text bug, we removed letterSpacing because we use maxLines 1 or 2 in a bunch of places with elipssis and the letterspacing ruins our design because of the android bug

[Stylianos Gakis]

Got any repro of this Colton? We're using the same and I haven't noticed it before

[Siyamed]

I believe i fixed a letterspacing, ellipsis and one more thing not playing well each other in compose.

[Colton Idle]

@Stylianos Gakis yeah I think the repro was pretty simple. I think i filed like 2+ bugs about it that @Siyamed said will have to be fixed in Android 13 or 14 since it was an OS level issue with Text. let me look up the issue tracker.

[dorche]

If you have a visual of the issue you can share it would be great too, sounds like I'll hit this issue soon too so I'd rather get ahead of it 😄