Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IOAPPX-326] Fix bug on Android when using SemiBold font weight #293

Merged
merged 2 commits into from
Jun 19, 2024

Conversation

dmnplb
Copy link
Collaborator

@dmnplb dmnplb commented Jun 19, 2024

Short description

This PR fixes a critical UI bug on Android when using the SemiBold font weight. The new Titillium Sans Pro uses Semibold as a font attribute, while the previous Titillium Web fonts used SemiBold. Android didn't handle this edge case and rendered everything with the fallback system font.

List of changes proposed in this pull request

  • Replace SemiBold font weight attribute with Semibold in all the typographic styles
  • Temporarily update MDH2 with a different font size to distinguish it from MDH3

How to test

  1. Launch the example app with an Android physical or virtual device
  2. Check Typography screen

@dmnplb dmnplb added the bug Something isn't working label Jun 19, 2024
@dmnplb dmnplb requested review from a team as code owners June 19, 2024 13:00
@dmnplb dmnplb changed the title [IOAPPX-326] Fix bug on Android when using Semibold font weight [IOAPPX-326] Fix bug on Android when using SemiBold font weight Jun 19, 2024
@dmnplb dmnplb merged commit 9231d82 into main Jun 19, 2024
6 checks passed
@dmnplb dmnplb deleted the IOAPPX-326-fix-weird-bug-android-semibold-font-weight branch June 19, 2024 13:53
dmnplb added a commit to pagopa/io-app that referenced this pull request Jun 20, 2024
…t weight (#5878)

> [!warning]
> This PR depends on:
> * pagopa/io-app-design-system#293

## Short description
This PR fixes a **critical** UI bug on Android when using the `SemiBold`
font weight. The new _**Titillium Sans Pro**_ uses `Semibold` as a font
attribute, while the previous _**Titillium Web**_ fonts used `SemiBold`.
Android didn't handle this edge case and rendered everything with the
fallback system font.
  
## List of changes proposed in this pull request
- Replace `SemiBold` font weight attribute with `Semibold` in all the
typographic styles

## How to test
1. Launch the app with an Android physical or virtual device
2. Check **Typography** screen or everything else in the app

---------

Co-authored-by: Cristiano Tofani <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants