Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

css (grid , flex and media queries) and html

HussinSoli 290

@HussinSoli

Desktop design screenshot for the Stats preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


did i over code this or it is fine, plz give me feedback if u can .. thanks in advance :)

Community feedback

@AGutierrezR

Posted

It seems that you use 'Lexend Deca' on <p>, and that is the reason for the lead text to be different from the design (Discover the bene....), Companies, Template, and Queries use 'Lexend Deca' and Upper

On the other hand, don't use <div> for the stats, you could use ul>li tags and inside use a span tag, this span could wrap the number of the stat description (companies, template, queries), so you don't have to rely on <h2> and <p> for that

Marked as helpful

2

@AGutierrezR

Posted

In order to fix the accesibilies issues, you could use sematics tags line like <footer class="attribution"> instead of <div class="attribution"> and wrap <div class="container"> with a <main> tag

Marked as helpful

1

HussinSoli 290

@HussinSoli

Posted

@AGutierrezR ok I will try it next time, thanks ^^

1
HussinSoli 290

@HussinSoli

Posted

very helpful , thanks alot !

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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