adeo / mozaic-design-system

Mozaic Design System
https://mozaic.adeo.cloud
Apache License 2.0
71 stars 17 forks source link

The MultiBanco icon is not visible enough #1381

Closed EddieVallier closed 1 year ago

EddieVallier commented 1 year ago

I am opening an issue for

Icon set

Package version

@mozaic-ds/vue@0.39.0

Description

We can't see the multibanco icon very well. image

GitHub repository

No response

Mock-up(s)

No response

What happened?

No response

What is expected?

No response

To Reproduce

Go to : https://ccdp.uat.leroymerlin.pt/checkout/cart

Code example

No response

Is this a regression?

Additional comments

No response

tiloyi commented 1 year ago

Hi @EddieVallier , Thanks for your issue.

As seen together, this is not actually a problem with the icon, but rather with the structure of this area.

Indeed, on LMFR, means of payment & payment services are on separate lines.

See this Figma mockup: => https://www.figma.com/file/Jxw480w4TnS90xoW2nqQK6/%F0%9F%92%A0-Library---Checkout?node-id=15%3A1450&t=eo11dnz5ohekzwu6-0

So, it would probably be necessary to standardize this, but I leave it to you to see with the Lead UI of the CCDP.

See You ✌🏼