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

chore(Cross): [IOAPPX-362,IOAPPX-364] Replace TypographyFactory with IOText + Add support for Bold Text a11y setting #6076

Merged
merged 111 commits into from
Oct 21, 2024

Conversation

dmnplb
Copy link
Contributor

@dmnplb dmnplb commented Jul 31, 2024

Short description

This PR replaces the typographic styles generated through useTypographyFactory with the new generic IOText typographic style. It also adds the support for the Bold text a11y setting to all the existing typographic styles.

List of changes proposed in this pull request

  • Update io-app-design-system to 2.0.0
  • Replace LabelLink with the relative Label… styles with the new asLink prop
  • Update DSTypography page to reflect all the latest changes
    • Remove MdH4, MdH5 and MdH6 typographic styles because not necessary

Preview

bold-text.mov

How to test

To define

@dmnplb dmnplb added a11y Activities related to accessibility Design System New visual language and reduction of previous UI clutter labels Jul 31, 2024
@pagopa-github-bot pagopa-github-bot changed the title [IOAPPX-362] Replace TypographyFactory with IOText + Add support for Bold Text a11y setting chore(Cross): [IOAPPX-362] Replace TypographyFactory with IOText + Add support for Bold Text a11y setting Jul 31, 2024
@pagopa-github-bot
Copy link
Collaborator

pagopa-github-bot commented Jul 31, 2024

Warnings
⚠️

Multiple stories with different types are associated with this Pull request.
Only one tag will be added, following the order: feature > bug > chore

Affected stories

  • ⚙️ IOAPPX-362: Aggiunta di IOText al posto degli stili tipografici generati con useTypographyFactory
    subtask of
  • ⚙️ IOAPPX-364: [io-app] Aggiunta del supporto dell'impostazione Testo in grassetto
    subtask of

Generated by 🚫 dangerJS against 9a418d8

@dmnplb dmnplb changed the title chore(Cross): [IOAPPX-362] Replace TypographyFactory with IOText + Add support for Bold Text a11y setting chore(Cross): [IOAPPX-362, IOAPPX-364] Replace TypographyFactory with IOText + Add support for Bold Text a11y setting Jul 31, 2024
@dmnplb dmnplb added Dark mode 🌝 and removed code review needed 💻📱 Insert this label if you need a code review that no one in your team can give you labels Oct 17, 2024
Copy link
Contributor

@CrisTofani CrisTofani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dmnplb dmnplb merged commit caf8138 into master Oct 21, 2024
13 checks passed
@dmnplb dmnplb deleted the IOAPPX-362-add-IOText branch October 21, 2024 13:19
dmnplb added a commit that referenced this pull request Nov 13, 2024
…typographic styles, associated with the previous UI (#6299)

## Short description
This PR removes legacy components related to the previous UI, including
local typographic styles. This move is necessary because we need to get
rid of the legacy typography to avoid typographic inconsistencies [once
the `IOText` component is
introduced](#6076).

## List of changes proposed in this pull request
- Remove the following local typographic styles: `Body`, `H1`, `H2`,
`H3`, `H4`, `H5`, `Monospace`, `Link`, `Label`, `LabelSmall`,
`BaseTypography`, `ComposedBodyFromArray`
- Remove `RobotoMono` typeface and its related references
- Remove local `TypographyFactory`
- Remove the following legacy components: `AdviceComponent`,
`RawAccordion`, `DetailedListItemComponent`, `InfoBox`
- Remove the following pages: `DSLegacyAccordion`, `DSLegacyTypography`

## How to test
Navigate through the app and check that everything is OK. Keep in mind
that some slight changes are normal because there isn't a perfect match
between legacy and new typographic styles.

---------

Co-authored-by: Emanuele Dall'Ara <[email protected]>
dmnplb added a commit that referenced this pull request Nov 22, 2024
…#6285)

> [!important]
> This PR is not directly dependent on the following PR. However, it
needs the new `IOText` to avoid unnecessary extra work:
> * #6076

## Short description
This PR extends dark mode support to some parts of the app

## List of changes proposed in this pull request
- **General**
  - Adapt `LoadingSpinnerOverlay`
- **Payments**
  - Adapt the payment biz event detail
  - Fix `ListItemTransaction` partial dark mode support
- **Settings**
  - Adapt the fiscal code screen
- Remove unnecessary `ContentWrapper` (by adding a new prop to
`IOScrollViewWithLargeHeader`)
- **Services**
- Adapt the detail screens (footer actions not included because they
need a general refactor)
- **Messages**
  - Adapt `MessageDetailsFooter`

## How to test
Run the app in the local environment, enable **Dark mode** and navigate
through the entire app, in particular the screens mentioned above

---------

Co-authored-by: Emanuele Dall'Ara <[email protected]>
Co-authored-by: Alessandro <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Activities related to accessibility Cross Dark mode 🌝 Design System New visual language and reduction of previous UI clutter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants