iam waiting for your feedback , feel free .

Community feedback

• 

  Maverick•20

  @PatrykGrzybPosted about 1 year ago

  From my side cause I am newbie and from User expierience I would say that input for e-mail could have diffrent coursor cause hands suggest that is a button.

  Marked as helpful

  0

  ---

  

  Sameh•430

  @SamehCodePosted about 1 year ago

  @PatrykGrzyb Thanks for your pass, i will try to get it as close as be

  0

  

  Sameh•430

  @SamehCodePosted about 1 year ago

  @PatrykGrzyb but if u look at active states image you'll find that hover on input email cursor pointer , but i modified it anyway. thank you :P

  0
• 

  Aaron Ragudos•480

  @RagudosPosted about 1 year ago

  Hello! I am new to Frontend Mentor, and my first try is awesome! Anyway, I stumbled upon your work and decided to check it out.

  My recommendations:

  • use <picture> element for images.
  • use a regex for email validation since invalid emails are still submitted.
  • use background-image CSS declaration to achieve the desired hover effect (gradient) on the button.

  All-in-all. Great job!

  Marked as helpful

  0