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

responsive layout page using grid

M7MOZ 20

@M7MOZ

Desktop design screenshot for the Four card feature section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

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

P
Nico 330

@Nico243

Posted

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
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