HTML, CSS, &Sass

Community feedback

• 

  P

  Ken•4,915

  @kens-visualsPosted about 3 years ago

  Hey @LonelyBuddy 👋🏻

  I have some suggestion for the project.

  • First, instead of separately giving border-radius to each card, you can do the following:

  main {
      border-radius: 0.5rem;
      overflow: hidden;
  }
  

  • For the car icons, add aria-hidden="true”, because they're for decoration. You can read more about aria-hidden here.
  • Also, I suggest adding transition: all 0.2s; to the button and the links, this will make :hover smoother.

  I hope this was helpful 👨🏻‍💻 overall, you did a great job, well done. Cheers 👾

  Marked as helpful

  1

  ---

  

  Henry Zheng•420

  @LonelyBuddyPosted about 3 years ago

  @kens-visuals Thanks for the feedback! The transition on buttons really makes the difference 😁.

  0

  

  P

  Ken•4,915

  @kens-visualsPosted about 3 years ago

  @LonelyBuddy no problem 😇

  0