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

Basic website using CSS Grids

aaditya78 60

@aaditya78

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


I am new to web development there are a lot of areas to be furnished in this, its the first time i am using CSS grids

Community feedback

Adarsh Pratap 5,515

@adarshcodes

Posted

Hi! @aaditya78, Welcome to this amazing community🙂 and Congratulations on completing your first solution of Frontend mentor. You did nice work on your first solution.👍 Let's take a look at your solution

  1. Design aspect
  • As @tomboynotes mention above about the heading fix it.
  • You can decrease the width of the card a little.
  1. Responsive aspect
  • Add some margin to your Team builder's card's icon, it is overflowing at breakpoint about 1082px around.
  • When you decrease down the window you'll need to add margin-bottom to all your icons.
  1. Code aspect
  • Your code looks fine
  1. Issues aroused
  • Amazing you've no issues aroused on your first solution, great👍.

You are doing great on your first solution🐱‍🏍. Keep Learning and Keep Coding🐱‍💻

1

@aadityarajkumawat

Posted

@adarshcodes, Thank you for suggesting all these, it helps me gain confidence and move towards being a productive web developer, I will get them rectified soon.

0

@shubhamthedev

Posted

Hi @aaditya78 the grids look awesome and the positioning is great and the site is responsive all the way. Here are some suggestions i would make:

  • The font sizes for the headings seems to be wrong, you should give hierarchies in design i.e. some things on the page are more important than others and should be visible first.

  • At around 400px the images seem to be touching the bottom of the boxes maybe give them a bottom margin.

That's all, keep working hard and keep coding ☺️.

1

@aadityarajkumawat

Posted

@tomboynotes ,Thanks a lot for pointing out some, corrections I will rectify them soon.

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