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

Yathish 50

@yathishg

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

Solution retrospective


Hello everyone, Accomplished second challenge. pls let me know if any mistake in the code and any suggestions. Thank you

Community feedback

xmat 160

@mati0165

Posted

Hi, nice that you have done that challenge, but i have some suggestions for you:

  • The best way to create a page is using divs not table like you do in that challenge. Also for that i would prefer using grid.
  • The page is not centered, but the best way to do that is when u use flexbox and add in css code min-height: 100vh for your body.

Marked as helpful

0

Yathish 50

@yathishg

Posted

@mati0165 thanks for your feedback but I had another doubt can u pls tell me what is that two warnings showing in accessibility report

0
xmat 160

@mati0165

Posted

@yathishg Yes, its because u didint use <main> or/and <footer> in your code. There are a few landmarks in HTML5. When u add <main> after body those errors should disappear. Try to rebuild your code using landmarks and divs for placement on the site

Marked as helpful

0
Yathish 50

@yathishg

Posted

@mati0165 got it ,thnks

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