Skip to content
Submitted about 5 years ago

Grid CSS, HTML, JS

Rafael Castellanos-Welsh•210
@rafawelsh
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Base Apparel coming soon page
  • HTML
  • CSS
  • JS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Just wondering what little tips y'all have to make this as clean as possible. thank you!

Community feedback

  • Matt Studdert•13,611
    @mattstuddertPosted almost 5 years ago

    Hey Rafael, great work on this challenge. My only couple of recommendations would be:

    • Make sure you have a h1 on the page. You should always have one h1, which will declare the main page heading.
    • One other approach to the displaying of the images would be to use the srcset attribute on the img tag. This allows for the conditional rendering of responsive images without the need for additional CSS.

    Keep up the great 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