3 Column preview card using HTML and CSS.

Community feedback

• 

  Jacek•95

  @sab0taszPosted over 3 years ago

  Hi! Good job on the challenge, however couple things need polishing.

  1. I don't think you actually need .main container. I think that .main-container will be enough. You can then use display: grid and place-items: center; on it to place it in the middle. At least that's the way I do most often.
  2. You did not put border-radius on the container.
  3. I would suggest not defining heights and widths for the specific cards. Make it so the content of the card determines it's size. Use margins and padding. Only one thing I did in my challenge was setting max-width on the container so it does not stretch to far.

  I hope it will help you somehow. Feel free to ask other questions, maybe I will be able to answer :D

  Cheers!

  0

  ---

  

  Jacek•95

  @sab0taszPosted over 3 years ago

  4. Oh and more thing. You should also put min-height: 100vh; on the body tag. It should enable somewhat responsiveness when you resive the window.

  0

  

  Mad_monkey•75

  @sugi13Posted over 3 years ago

  @sab0tasz thank for your feedback buddy :)

  0

  

  Mad_monkey•75

  @sugi13Posted over 3 years ago

  @sab0tasz It was my first project..buddy.I keep doing great.. :) thank you for your feedback.

  0

  

  Jacek•95

  @sab0taszPosted over 3 years ago

  No worries, glad I could help, I hope this is actually the thing that should help you. Let me know when you make the changes, if they worked ;)

  0

  

  Mad_monkey•75

  @sugi13Posted over 3 years ago

  @sab0tasz ok buddy

  0