3 column card

Community feedback

• 

  Remus D. Buhaianu•3,125

  @remusbuhaianu

  Posted over 3 years ago

  Congrats on completing the challenge, @PitagoraStrike!

  I had a look at your final solution and Github repo. Here are a few suggestions that might help:

  • On larger screen sizes, there's a bit too much space at the top

  • Responsiveness needs a bit of polishing - you have horizontal scrolling on smaller screen sizes

  • Don't use fixed width and height, especially not with the px unit

  • You've used a lot of absolute positioning, which is really not needed in this case. You can build simple, modern, and efficient layouts and position your elements properly using Flexbox or CSS grid

  01

  Marked as helpful
• 

  darryncodes•6,350

  @darryncodes

  Posted over 3 years ago

  Hi Victor,

  Good solution, nice work!

  A few minor bits of feedback for you:

  • it might be cool to add some border-radius to your design to match the challenge
  • you could add transition: ease .3s; to your buttons to make the hover affect smoother
  • it's best practice to avoid setting fixed heights and widths for responsive design, the content usually dictates this. i'd recommend looking into this further. this is a really good free course

  Keep up the hardwork!

  01

  Marked as helpful