Product Preview

Community feedback

• 

  Pranshu Sahu•770

  @Pranshu-SahuPosted about 2 years ago

  🎉Hello @Msarthaksharma, congratulations on your solution!

  ✨ Nice code and nice solution! You did a good job here putting everything together. I’ve some suggestions for you:

  1- Center the container inside the body , body{ display:flex; justify-content:center; alignt-items:center; min-height:100vh; } Remove the positioning you have given to .container

  2- add max-width:35rem to container this would make your container fluid, instead of giving any fixed width.

  3- Remove border from button, add hover effect to improve .btn { border:none; } .btn:hover{ translate:1px 1px; opacity:0.6; }

  4- You have responsive design, in order to make to make it responsive :- .container{ display:flex; justify-content:center; align-items:center; @media only screen and (max-width: 600px) { .container{ flex-direction:col; }

  Make image and card text content two flex items.

  Hey sarthak, working on projects which looks quite simpler may be quite frustrating 🙇‍♀️ and confusing🤦‍♂️ in initial stages. 🎇 Have faith in yourself, I'm sure after that after working on couple of such project, your confidence would be high.

  You would be surely succeed 🏆 and would achieve your dreams.

  Happy coding 👨‍💻

  Feel free to leave comment if you need any further help.

  I hope this would be helpful.

  Marked as helpful

  0

  ---

  

  Sarthak Sharma•50

  @MsarthaksharmaPosted about 2 years ago

  @Pranshu-Sahu hi Thank you so much for your feedback, I would love to implement it.

  0