HTML and CSS Solution

Community feedback

• 

  ~ Ammar•100

  @andernaderPosted about 2 years ago

  Hey Akshat, awsome work!

  It seems you forgot about the footer element out there, it is sitting at the middle of the page XD. well, I have two suggestions for you :

  • For the div element that you wrapped your image with it : what do u think about applying a display:flex with align-items: flex-startto the whole card section then only applying align-self: flex-end to the image itself, wouldn't that be better ?
  • The other thing is that it seems your page isn't fully responsive, what do you think about using grid on the whole card sections, you can wrap them inside a div with display grid and then changing the grid-template-columns value depending on the width of the device with media query.
  • Ah , last thing i would love to mention I told from vcarames here is that you should start your design with mobile design first then moving on to adjust it for desktop devices , you can search more if about that if you are intersted.

  i hope I was helpfull, have a great day ~

  1

  ---

  

  Akshat Vishwakarma•240

  @vishwa-akshatPosted about 2 years ago

  @andernader Thank you for the suggestions. I find them really insightful and I will make some changes on this project.

  1