Frontend Mentor - 3-column preview card component

Community feedback

• 

  PhoenixDev22•16,950

  @PhoenixDev22Posted about 2 years ago

  Hello Luis-Olivero,

  Congratulation on completing this challenge. Excellent work! I have few suggestions regarding your solution, if you don't mind:

  HTML

  • Page should have at least one level heading. You can have<h1>visually hidden with sr-only.

  • Adding rel="noopener" or rel="noreferrer" totarget="_blank"links. When you link to a page on another site using target=”_blank”attribute , you can expose your site to performance and security issues.

  • Add border-radius andoverflow hiddento the main container that wraps the three cards so you don't have to set it to individual corners.

  Aside these, great job on this one. Hopefully this feedback helps.

  Marked as helpful

  0
• 

  Cristhian•120

  @Cosenza26Posted about 2 years ago

  I would recommend to add some padding after each description

  Marked as helpful

  0