Responsive four card feature using CSS Grid

Community feedback

• 

  Jordane Gengo•590

  @jgengo-altPosted over 2 years ago

  Hello there my dear friend 👋

  First of all, great job 🚀

  On a wide-screen your layout is a bit too spacy, would have been great to add a max-width of 1200px to your parent div

  Also it would have been more intuitive to add position: relative to the card, and position: absolute; to the image so you can place it in space with more "relatable" position like bottom: 2em; right: 2em;

  Once again you did tho a great job!

  Keep going 💪

  Don't hesitate to follow me and give me some feedbacks on my assignments, I will follow back and try my best to give you feedbacks to your future posts :) 🪴

  Marked as helpful

  0

  ---

  

  P

  Graeme Byrne•520

  @grmbyrnPosted over 2 years ago

  @jgengo-alt Some really good advice there, Jordane! I'm sure they'll help improve the page. I just followed you. I'll give feedback where I can!

  0
• 

  omardavidcyber•295

  @omardavidcyberPosted over 2 years ago

  The only nit-picky thing is the page background color is white while the design is a more grayish one. But other than that the layout looks really good Graeme!

  Marked as helpful

  0

  ---

  

  P

  Graeme Byrne•520

  @grmbyrnPosted over 2 years ago

  @flyhighparadise Ah, well spotted. I didn't pick up on the background when looking at the Figma. Thanks!

  0