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

Four-card-feature-section-master

@abderrahmanebnd

Desktop design screenshot for the Four card feature section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Mirjax2000 680

@Mirjax2000

Posted

page icon karma, i noticed. Nice one.

Html - too many divs. Use other elements with purpose. Header,main,footer,aside,section,article and many others. It helps browser better understand what is he displaying, also good for readers and also it helps better evaluate your page with SEO. Then you have better score and your site will be displayed higher.

--root colors:

:root {
    /* Primary  */
    --Red: hsl(0, 78%, 62%) ;
    --Cyan: hsl(180, 62%, 55%) ;
}

What happened if your designer or contrator or you will decide that color hsl(0, 78%, 62%) have to change to green? so your variable --red: will be green. And that is a mess. use something neutral for that, for example

:root {
    /* Primary  */
    --header: hsl(0, 78%, 62%) ;
    --paragraph: hsl(180, 62%, 55%) ;
    --border-calculator:blue;
    --border-karma:yellow;
    --border-superviser: darkgreen;
    --border-teambuilder:red;
}

I like your flexbox styling.

see you

Marked as helpful

0

@abderrahmanebnd

Posted

hey Mirjax, i liked the idea of changing the names of variables , but i just copied it from the given style file , Thanks bro.

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