Four card feature section (HTML,CSS with FlexBox) + Responsif

Community feedback

• 

  Greta•340

  @gretagrPosted almost 5 years ago

  Hey, congrats on your first solution! I think your code is actually quite neat! You use HTML5 tags really nicely! (maybe you don't need the div's around article tags tho). However, I have some thoughts about the design itself:

  1. Your cards are way too big.
  2. There is no spacing around cards when the screen gets smaller.
  3. Heading text overlaps text below it on a mobile screen, and cards do not fit the screen.
  4. As mentioned before - you forgot to use given fonts.

  Now, it might seem to be not so much of an issue (the code is good, right?), but as a frontend developer, your goal is to get the website looking as close to the design as possible, so I thought this kind of feedback might be useful too :)

  Good luck on your next projects, you're off with a good start!

  1

  ---

  

  avisikhsan•20

  @avisikhsanPosted almost 5 years ago

  @gretagr thank you for the correction and advice

  0
• 

  Axel•150

  @Axel01011111Posted almost 5 years ago

  Hey, it looks great. You could check the fonts and the title part on the mobile design, but everything else is good in my opinion

  1
• 

  Ivan Setiawan•30

  @gadroxjrPosted almost 5 years ago

  To be honest, I prefer sans serif type fonts, like Roboto or others. You can find alternative fonts from Google Fonts. The shadow on each card is also too thick. You can reduce the opacity so it's not too thick. Good job for your first challenge! Keep going!

  1