Responsive 3-column grid preview card component

Community feedback

• 

  Kamasah-Dickson•5,590

  @Kamasah-DicksonPosted about 2 years ago

  Your solution looks great but i want to help you with some small tips.

  1 . Do not specify any height in the grid items. This sometimes breaks layout and might stress you some other time when things are not working as expected. Instead allow the items to determine their own height. What i mean is that paddings and margins applied to the grid items will automatically determine the heights of the grid items. I hope you get me.

  2 . To help you with the accessibility issues, remove the attribution class and change the div to a footer

  Do not use a section tag if there won't be any direct head tag I mean h1....h6 always use a section tag if there would be a directed head tag in your page otherwise use a div instead.

  Good job there👍🎉 Happy coding

  Marked as helpful

  1

  ---

  

  Konjeti Maruthi•70

  @MaruthiKoPosted about 2 years ago

  @Kamasah-Dickson, Thank you so much for your valuable info, will do the necessary changes

  1