Fylo landing page

Community feedback

• 

  ronaldlamdev•430

  @ronaldlamdevPosted over 2 years ago

  Great job on coding this challenge. There is one thing I would change with your email form. I would change the input for your email to text in your HTML. For example, I would change <input type="email"/> to <input type="text"/>. This will remove the default error message (not the red error message), but it'll make your form error cleaner and easier to read. Having two error messages is kinda redundant.

  I would also start using more id attributes in your HTML instead of repeatedly using class to name your tags. The benefit of this is that id attributes allow links to navigate to other sections of your code. For example, instead of class = "features", use id = "features". As for <li> features </li>, use <li href="#features"> features </li>. This way, the site itself will automatically scroll to the feature section of your site instead of having the user manually scroll to the site.

  Marked as helpful

  1

  ---

  

  Daniela Darnea•180

  @mdanieladlaPosted over 2 years ago

  @ronaldlamdev thank you for the feedback :)

  0