Imposible to align custom font [235876330]

Fixed

Feature Request

Status Update

No update yet.

Description

iv...@bandlab.com

created issue #1

Jun 14, 2022 02:40AM

We use a custom font (developed by 3rd party company) in our app, and we found that by using the recently introduced text style setting includeFontPadding=false we cannot center text properly (see the attached screenshot).

Example:

Text(
    text = "Tough Bounce",
    color = Color.Black,
    style = TextStyle(
        fontSize = 12.sp,
        lineHeight = 16.sp,
        fontFamily = null, // custom font family here
        platformStyle = PlatformTextStyle(
            paragraphStyle = PlatformParagraphStyle(includeFontPadding = false),
            spanStyle = null
        ),
        lineHeightStyle = LineHeightStyle(
            alignment = LineHeightStyle.Alignment.Center,
            trim = LineHeightStyle.Trim.None
        )
    ),
)

More information in this thread: https://kotlinlang.slack.com/archives/CJLTWPH7S/p1654746582388999?thread_ts=1617664367.220300&cid=CJLTWPH7S

Jetpack Compose version: 1.2.0-beta03
Jetpack Compose component used: BasicText/Text, TextStyle
Android Studio Build: #AI-213.7172.25.2211.8624637
Kotlin version: 1.6.21
Devices/Android versions reproduced on: any
Keyboard (i.e. Gboard, Samsung, etc): -

fonts.jpg

319 KB

View

Download

Comments

si...@google.com <si...@google.com> Jun 14, 2022 04:04PM

Reassigned to si...@google.com.

si...@google.com <si...@google.com> #2Jul 24, 2022 02:49PM

Hi. Thanks for reporting this. Fixed in this changelist. This should be available in alpha-04

si...@google.com <si...@google.com> #3Jul 30, 2022 03:16AM

Project: platform/frameworks/support
Branch: androidx-main

commit e782987543a9f8ccd485e970ddc74564b24378db
Author: Vighnesh Raut <vighnesh.raut13@gmail.com>
Date: Mon Jan 02 15:27:40 2023

fix: tab row crashes when only 1 tab is added

Bug:

b/264018028

Test: Added unit test

Change-Id: I6381dbac304fc1d69d3708c6655f8b595668e93f

M tv/tv-material/src/androidTest/java/androidx/tv/material/TabRowTest.kt
M tv/tv-material/src/main/java/androidx/tv/material/TabRow.kt

https://android-review.googlesource.com/2373449

Message last modified on Jul 30, 2022 03:17AM

si...@google.com <si...@google.com> #4Aug 1, 2022 06:03PM

deleted

si...@google.com <si...@google.com> #5Aug 1, 2022 06:04PM

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.tv:tv-material:1.0.0-alpha04

Message last modified on Aug 2, 2022 01:06AM

Mon Aug 01 2022 11:04:05 GMT-0700 (Pacific Daylight Time).png

355 KB

View

Download

si...@google.com <si...@google.com> #6Aug 1, 2022 06:05PM

I will consider making the constructor public however since the whole system is based on those numbers in the font, I think it is better to update the font if it was possible. (i.e. using fontTools link)

si...@google.com <si...@google.com> Aug 3, 2022 12:49AM

Marked as fixed.

ap...@google.com <ap...@google.com> #7Aug 3, 2022 12:49AM

Project: platform/frameworks/support
Branch: androidx-main

commit eea55e5a6d2f6e243dc0728403d293c79e5a9bda
Author: siyamed <siyamed@google.com>
Date: Mon Aug 01 10:53:31 2022

Make LineHeightStyle.Alignment constructor public

Some font ascent/descent/ymin/ymax values are not "correct".
for example ascent goes over ymax; or descent goes below ymin.

This CL makes the alignment constructor public so that fine tuning
for cases such as center is possible.

RelNote: "LineHeightStyle.Alignment constructor is now
public (experimental)"

Test: Existing tests already cover
Bug: 235876330
Change-Id: I4bbbe53b2088b9b211b86919c55bf6be88572baf

M compose/ui/ui-text/api/restricted_current.txt
M compose/ui/ui-text/src/commonMain/kotlin/androidx/compose/ui/text/style/LineHeightStyle.kt
M text/text/src/main/java/androidx/compose/ui/text/android/style/LineHeightStyleSpan.kt
M compose/ui/ui-text/api/public_plus_experimental_current.txt
M text/text/src/androidTest/java/androidx/compose/ui/text/android/TextLayoutTest.kt
M text/text/src/androidTest/java/androidx/compose/ui/text/android/style/LineHeightStyleSpanTest.kt
M compose/ui/ui-text/api/current.txt
M compose/ui/ui-text/src/androidMain/kotlin/androidx/compose/ui/text/platform/extensions/SpannableExtensions.android.kt

https://android-review.googlesource.com/2169775

ai...@gmail.com <ai...@gmail.com> #8Aug 4, 2022 05:30AM

Thanks a lot for those changes. We will try to fix font with fontTools, but I think having Alignment constructor public still can be useful for fine tuning of the font