grid and flexbox

Community feedback

• 

  Aayush Sood•1,175

  @soodaayushPosted almost 4 years ago

  Your website is good! But there are things you can improve on:

  1. As grace-snow said, the text is darker than the design

  2. The font size of the people's names is a bit too large. Same with the header beneath that.

  3. I noticed that there were no shadows around the boxes.

  That's all I have to say!

  Happy Coding!

  0

  ---

  

  Abdul•255

  @abdulrahman-rwimPosted almost 4 years ago

  @WarBaddy thanks for your feedback I really preached it.

  0
• 

  T

  Grace•31,270

  @grace-snowPosted almost 4 years ago

  Really nice job on this. Some of your text is darker than the design, but that's a minor thing.

  Not sure you need to wrap images in a div either, but that's just a preference thing.

  Well done, nice clear html and css

  0

  ---

  

  Abdul•255

  @abdulrahman-rwimPosted almost 4 years ago

  @grace-snow thank you so much for your feedback that was really helpful I'm not sure if my flexbox gonna work without a wrap div so I used to wrap everything when I want to use flexbox.

  0