3 column preview card component

Community feedback

• 

  Cheosphere•1,040

  @CheospherePosted about 2 years ago

  @agelitaeme That's an excellent job!... maybe you think it's a good idea to apply a transition effect to the "Learn More" buttons, to make the hover effect look smoother, you can apply the following line in your code:

  .card a {
    width: 150px;
    height: 50px;
    border-radius: 30px;
    text-decoration: none;
    background-color: var(--gray);
    display: flex;
    justify-content: center;
    align-items: center;
    transition: all .25s ease-in-out;
  }
  

  A hug and continue happy coding 😁

  Marked as helpful

  2
• 

  バレンタイン 😈•64,170

  @VCaramesPosted about 2 years ago

  Hey @agelitaeme, some suggestions to improve you code:

  • The car images/icons serve no other purpose than to be decorative; They add no value. Their Alt Tag should left blank and have an aria-hidden=“true” to hides it from assistive technology.

  • The headings are being use incorrectly. For this challenge, each heading is equally as important. So best option, is to use <h2> Heading, because it will give each card the same level of importance and it's reusable.

  • Implement a Mobile First approach 📱 > 🖥

  With mobile devices being the predominant way that people view websites/content. It is more crucial than ever to ensure that your website/content looks presentable on all mobile devices. To achieve this, you start building your website/content for smaller screen first and then adjust your content for larger screens.

  Happy Coding! 👻🎃

  Marked as helpful

  1