Skip to content
Submitted 3 months ago

responsive layout page using grid

M7MOZ•20
@M7MOZ
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Four card feature section
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

Using CSS grid was a challenge to me in the begining but now I am more familiar with it

What specific areas of your project would you like help with?

i want to know if my query for responsive design is good

Community feedback

  • Nico•330
    @Nico243Posted 3 months ago

    You have a great code that can use some minor improvements.

    The font statements includes many fonts which makes the loading time slower. If you're only going to use poppins use only that specific font to reduce load time.

    Try to use <h1> tags for the following e.g. <p class="mid">Reliable, efficient delivery</p>, this will improve accessibility and and SEO.

    Make sure to remove or update comments in the future.

    You might want to add alt="#" to your <svg> tags to describe the images for viewers on desktops or mobiles.

    Happy Coding!

    Marked as helpful
    1

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