Four card feature section

Community feedback

• 

  bikeinman•1,080

  @BikeInManPosted over 2 years ago

  The design on desktop seems to be very close to the design. Congratulations.

  I have the following comments for you, if you will.

  You should not use div to wrap text. Div is mostly used to group of elements and to style it. .cardTitle, .cardP should be changed to standard html elements like h2 and p.

  Cards stretch the whole screen on smaller than desktop sizes. Try using max-width (about 300px) on your cards.

  You can make this challenge more interesting by adding some interactivity to the cards like a hover effect and also making each card a link. They are supposed to lead somewhere, right?

  If you can push yourself a little more, you may even try to display 2x2 cards on tablet sizes.

  Hope you enjoyed doing this challenge. For me, it took a lot of time and a lot of trial and error.

  Good Luck.

  Marked as helpful

  0

  ---

  

  Axurynn•200

  @AxurynnPosted over 2 years ago

  @BikeInMan Thank you so much for your feedback! I made this challenge a long time ago without posting it here :) I tried to improve it with your comments but it would be easier to restart it from scratch!

  Thanks again and happy coding ! 😄

  0