mozilla-mobile / firefox-ios

Firefox for iOS
Mozilla Public License 2.0
12.15k stars 2.91k forks source link

Update Fonts related to BreachAlertsDetailView to use FXFontStyles #20469

Open data-sync-user opened 3 months ago

data-sync-user commented 3 months ago

This task is part of a series of tasks to standardizing fonts to start using FXFontStyles.

Background Context: Previously, we were using DefaultDynamicFontHelper and setting the text style, size and weight for each font, which sometimes did not match our design system. By using FXFontStyles, we can standardize our fonts and be more aligned with the design system.

Task: Please update how we set fonts in the Firefox iOS Project related:

BreachAlertsDetailView

Acceptance Criteria:

Replace usage of DefaultDynamicFontHelper with FXFontStyles

If there are discrepancies in terms of replacing the old fonts with the new standard, we will need to bring design in to approve the PR.

Please provide before and after screenshots of the UI.

Reference:

Usage example: FXFontStyles.Regular.headline.scaledFont()

See this PR for example of standardizing fonts for Primary Button:

https://github.com/mozilla-mobile/firefox-ios/pull/18711/files

┆Issue is synchronized with this Jira Task

data-sync-user commented 3 months ago

➤ Tushar Chitnavis commented:

Attaching ‘Before’ and ‘After’ screenshots of the BreachAlertsDetailView. Please verify the fonts in red. Thank you!

!Screenshot 2024-06-20 at 6.23.59 PM.png|width=754,height=655,alt="Screenshot 2024-06-20 at 6.23.59 PM.png"!

data-sync-user commented 2 months ago

➤ Diana Andreea Barladeanu commented:

Hi Tushar Chitnavis How should we trigger a breach alert in order to test this ticket? Many thanks!

data-sync-user commented 2 months ago

➤ Tushar Chitnavis commented:

Hi Diana Andreea Barladeanu , for testing , Breach Alert can be triggered only via hard code.

I had made changes locally to trigger the alert. Do let me know if you need any help to verify. Thank you!

data-sync-user commented 2 months ago

➤ Diana Andreea Barladeanu commented:

Validated on v128(43182), with iPhone 15 (17.5).