Design comparison
Solution retrospective
Hi, Im aware that something is off with the typography of <h1> tag, if anyone could tell me how to fix this or have any other optimization suggestions, I'll be glad to hear you out!
Community feedback
- @correlucasPosted about 2 years ago
๐พHello Niklaus, congratulations for your new solution!
๐ฏ Your solution its almost done and Iโve some tips to help you to improve it:
1.Remember that you cannot have more than one h1 heading. So choose one of the cards to keep the
h1
and the others can be replaced with anh2
.You need to increase the headings by one level, like h1, h2, h3 to show the titles hierarchy.2.You made your html structure entirely with
div blocks
but these div doesn't any semantic meaning, for this reason is better you use a better html markup improving your code, for example for each vehicle card you use<article>
instead of the<div>
.2.Note that you need to add the rounded borders only to some borders and not al the sizes of each card. You can fix that adding rounded borders only for the left borders for the first card and the opposite for the third.
โ๏ธ I hope this helps you and happy coding!
Marked as helpful2@NiklausRupailPosted about 2 years ago@correlucas Thanks a lot! Everything you said is fixed now, but still I got no idea on how to make these headings like in the design
0@correlucasPosted about 2 years ago@NiklausRupail Import the correct font
font-family: 'Big Shoulders Display', cursive;
<link rel="preconnect" href="https://fonts.googleapis.com"> <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin> <link href="https://fonts.googleapis.com/css2?family=Big+Shoulders+Display:wght@800&family=Lexend+Deca:wght@200&display=swap" rel="stylesheet"> </head>
Marked as helpful1
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