Submitted 5 months ago
i made use of 100vw & 100vh for the background ,then run inspection
@waliuakindolapo
Design comparison
SolutionDesign
Community feedback
- @gabgonsan712Posted 5 months ago
I liked the design. You missed landmarks, the rest of the html semantic is good. You can use a normalize.css so you can delete the white margins in the page. It is not responsive on some screens. The solution is very similar to the design(very good) The page is accesible. In the css, instead of writing "font-family: Figtree;" for every item, you can use the tag * { font-family:Figtree;}, to save space, same case with the link for google fonts. For responsive, you can learn to use flex, as it is easy for responsive design.
Marked as helpful0@waliuakindolapoPosted 4 months ago@gabgonsan712 thanks a lot for the corrections and tips
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord