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

refactor: setup SwiftUI #58

Closed florentlotthepro closed 2 years ago

florentlotthepro commented 2 years ago

Changes description

Done all the setup to use SwiftUI in the project

Context

SwiftUI is a new Apple technology and will be used more and more so we need to be present on it.

Checklist

Does this introduce a breaking change?

Screenshots

iPhone

Simulator Screen Shot - iPhone 11 - 2022-04-25 at 15 30 06

iPad

Simulator Screen Shot - iPad (7th generation) - 2022-04-25 at 15 31 11

Other information

MARIEDELATTRE commented 2 years ago

Hi guys thanks for this PR. Here is my design review (I have checked in darkmode) : • Assets: maybe it's on our side, but I think favicon shouldn't be transparent but white in background + favicon-black seem to not be black. • Colors: it seems OK in darkmode please double-check with @simon if the last changes we made are ok like in borders. • Icons: it's a bit blurry on my iphone is it normal? • Radius: could you please display rectangles because I can't chek if 800 is ok because the radius is larger than the height of the frame (the last one seem weird). • Shadows: I saw that we don't add the shadows in darkmode like in the wel library it seems ok, but maybe you want us to add them in the design lib to verify? • Typography: bold equivalents are missing (footnote callout and body)

And it's details but the titles are sometimes plural sometimes singular I think it can be harmonized 🙂

sonarcloud[bot] commented 2 years 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