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

3-column-preview-card-using-grid

Shehab Sha'lanโ€ข 30

@shehabshalan

Desktop design screenshot for the 3-column preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Ahmed Bayoumiโ€ข 6,800

@Bayoumi-dev

Posted

Hey Shehab, It looks good!... You have accessibility issues that need to fix.

  • Documents must have <title> element to aid in navigation
<head>
   //...
   <title>3-column preview card component | Frontend Mentor</title>
</head>
  • Using more than one <h1> is allowed by the HTML specification, but is not considered a best practice. Using only one <h1> is beneficial for screenreader users.

---> Multiple <h1> elements on one page

I hope this is helpful to you... Keep coding๐Ÿ‘

1

Shehab Sha'lanโ€ข 30

@shehabshalan

Posted

@Bayoumi-dev Thanks. I will update my code.

0
Okafor Raphaelโ€ข 240

@OkaforRaph

Posted

Hey Shehab, congratulations on completing the challenge! You did a great job ๐Ÿ˜‰

Just a tip for optimizing your code:

  • Always make use of relative units of measurement like "rem" instead of px.

Hope this help! Happy coding ๐Ÿ˜

0

Shehab Sha'lanโ€ข 30

@shehabshalan

Posted

@OkaforRaph thanks for the tip.

0

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