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

[IOAPPFD0-167] Add the new LogoPaymentCard component #92

Merged
merged 3 commits into from
Oct 3, 2023

Conversation

dmnplb
Copy link
Collaborator

@dmnplb dmnplb commented Oct 3, 2023

Short description

This PR adds the new LogoPaymentCard, a component that is going to be used in the new payment cards.

List of changes proposed in this pull request

  • Add the new LogoPaymentCard with the following props:
    • height (default: 32)
    • width (default: 100%)
    • align: start, center, end
  • Update documentation in the Logos page

Preview

How to test

  1. Launch the example app
  2. Go to the Logos page

@dmnplb dmnplb requested review from a team as code owners October 3, 2023 12:25
@dmnplb dmnplb merged commit 962aba4 into main Oct 3, 2023
@dmnplb dmnplb deleted the IOAPPFD0-167-refactor-logo-payment-extended branch October 3, 2023 18:21
dmnplb added a commit to pagopa/io-app that referenced this pull request Oct 17, 2023
…Design System (#5077)

## Short description
This PR adds the documentation of the `LogoPaymentCard` to the `Logos`
DS page. To learn more, see the related PR:
* pagopa/io-app-design-system#92

## List of changes proposed in this pull request
- Update `io-app-design-system` to the `1.12.0`
- Add `LogoPaymentCard` to the relative DS page
- Update `DSComponentViewerBox` and `DSLogoPaymentViewerBox` components
to improve logo visualisation of the above component

### Preview
<img
src="https://github.com/pagopa/io-app/assets/1255491/64f71833-5003-4f79-ad0e-01439c280118"
width="320" />

## How to test
Go to the **Profile → Design System → Logos**
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants