Product preview card

Community feedback

• 

  Ahmed Bayoumi•6,740

  @Bayoumi-devPosted over 2 years ago

  Hey Ashutosh, It looks good!... Here are some suggestions:

  • Document should have one main landmark, Contain the component with <main>.

  <main>
     <div class="container flex-center">
        //...
     </div>
  </main>
  

  • Add cursor: pointer; to the button, The cursor indicates to users there is an action that will be executed when clicking on it.

  • Use REM for font size, It is a must for accessibility because px in some browsers doesn't resize when the browser settings are changed... See this article ---> CSS REM – What is REM in CSS?

  I hope this is helpful to you... Keep coding👍

  Marked as helpful

  0

  ---

  

  Ashutosh Bisoyi•20

  @ashutoshbisoyiPosted over 2 years ago

  @Bayoumi-dev Thank you for the helpful suggestions. I'll bring them to practice

  0