Created with Sass and Grid

Community feedback

• 

  Veronika Palto•305

  @JepardMayPosted over 3 years ago

  If there's a problem you throw an error, so add else to remove the error message if an email is correct (remove style properties and set innerHTML to an empty string again). It'll be good to add an email send by entering press (you can add a new keydown event listener or change button type to submit and listen to the submit event)

  Also, I notice that the mobile version is not usable on the small height - there's no scrolling, and some content's cut.

  Marked as helpful

  0

  ---

  

  Huseyn•85

  @Huseyn-SomeGuyPosted over 3 years ago

  @JepardMay thank you for your opinion. I will fix these problems

  0

  

  Huseyn•85

  @Huseyn-SomeGuyPosted over 3 years ago

  @JepardMay I think it should be fine now

  0

  

  Veronika Palto•305

  @JepardMayPosted over 3 years ago

  @Huseyn-SomeGuy it's better now, but header is not available on the mobile screen https://gyazo.com/5428243c5fae3cd15f5332bde23b0433 .The easiest way to correct it is to change height: 100vh on .wrapper to min-height: 100vh

  0

  

  Huseyn•85

  @Huseyn-SomeGuyPosted over 3 years ago

  @JepardMay ok will do it now.

  0
• 

  Abhishek Tiwari•30

  @itssabhishekPosted over 3 years ago

  Can you please share some ss of your problem?

  0

  ---

  

  Huseyn•85

  @Huseyn-SomeGuyPosted over 3 years ago

  @itssabhishek I don't know how to share ss over here :(

  0