Product Preview Card (responsive)

Community feedback

• 

  Andrey•2,550

  @dar-juPosted about 2 months ago

  Hi praises1!

  You did well! I liked how you connected the icon to the button using a custom attribute.

  • Look at the resolution 600-634 pixels, the image is not the full height of the block. I'm sure you know how to fix it )

  • For prices, I would not use the paragraph tag, span is more suitable.

  • As a recommendation: for .btn, use transition: background-color ease-in-out 0.3s; - this will give smoothness to the hover, interaction with active elements will be more pleasant for the user

  Otherwise, great work, good luck with the development!

  Marked as helpful

  0

  ---

  

  praises1•80

  @praises1Posted about 2 months ago

  @dar-ju Thank you for the feedback!

  0
• 

  P

  Habeeb Kareem•490

  @olaide-hokPosted about 2 months ago

  Kindly consider adding accessibility considerations Issue: The button hover effect changes the background color but doesn’t account for contrast or focus states.

  Fix: Add focus styles and ensure sufficient contrast for accessibility.

  .btn:focus {
      background-color: var(--clr-primary-500);
  }
  

  Marked as helpful

  0

  ---

  

  praises1•80

  @praises1Posted about 2 months ago

  @olaide-hok Thank you for the feedback!

  0