Skip to content
Submitted almost 3 years ago

Frontend Mentor - 3-column preview card component

Luis Olivero•170
@Luis-Olivero
A solution to the 3-column preview card component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Here is my second attempt after all your feedback! Thank you all for the help and advice given.

Code

Please log in to post a comment

Log in

Community feedback

  • PhoenixDev22•16,830
    @PhoenixDev22

    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
  • Cristhian•120
    @Cosenza26

    I would recommend to add some padding after each description

    Marked as helpful

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord