Decathlon / vitamin-ios

Decathlon Design System UI components for iOS & iPadOS applications
https://www.decathlon.design
Apache License 2.0
46 stars 14 forks source link

feat: add new icons #93

Closed florentlotthepro closed 1 year ago

florentlotthepro commented 1 year ago

Changes description

Added new icons.

Context

New icons were added in the Figma file.

Checklist

Does this introduce a breaking change?

Screenshots

iPhone

Capture d’écran 2022-10-28 à 21 35 53 Capture d’écran 2022-10-28 à 21 29 20 Capture d’écran 2022-10-28 à 21 29 27 Capture d’écran 2022-10-28 à 21 29 39 Capture d’écran 2022-10-28 à 21 29 58 Capture d’écran 2022-10-28 à 21 30 14 Capture d’écran 2022-10-28 à 21 30 27

Other information

daniel-dumortier commented 1 year ago

@florentlotthepro I did not have time to make a full review, but I had a feedback on icons from a developer about Vitamix icons.

It seems that when you try to display a Vitamix icon in a bigger size, it is pixelized. And that we could avoid that by checking "Preserve Vector Data" on each icon in assets catalog (in the right panel, in Image Set tab)

image

Since it relates to the same foundation, could we add this in this PR ?

sonarcloud[bot] commented 1 year ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

florentlotthepro commented 1 year ago

@daniel-dumortier Good point! I've done the update 🙂 And also for VitaminAssets.