Skip to content
Submitted almost 3 years ago

Responsive Landing Page using FlexBox and Grid

medghazaly•200
@med-ghaza
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Meet landing page
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Any suggestions on how I can improve are welcome!

Community feedback

  • Rio Cantre•9,690
    @RioCantrePosted almost 3 years ago

    Hello there! Awesome work with this one. Viewing at your solution, I would suggest the following for you...

    • Change the color for the --p-color into hsla(240, 10%, 57%, 0.25); and add color: hsl(240, 10%, 56.9%); for the number
    • Add description to this part <img src="assets/tablet/image-hero.png" alt="">
    • Wrap hero with specific tag like header and container with main tag for readability

    Overall you did well, hope this helps and Keep up the good work!

    0

Please log in to post a comment

Log in with GitHub

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