Fylo dark theme landing page

Community feedback

• 

  Mijail Hernandez•950

  @MishaHernandezPosted almost 4 years ago

  Hi Karim, I have some observations for you:

  • I recommend not using <br> tags to generate spacing between elements, instead use margins, padding or spacing via flexbox or grid.

  • Use <buttons> tags instead of <div> and use the font-size property instead of header tags.

  • In the action call (.start1) use a <form> tag as a container for the text box and the button. Use a padding in the main container and use margins instead of <br>.

  • In .social-media you use fontawesome icons but you must assign font css properties to give it size and color.

  Finally I recommend that you review documentation about semantic html. Greetings and continue with the challenges :)

  2

  ---

  

  Karim•810

  @Karimsamir112Posted almost 4 years ago

  @MishaHernandez okay thanks you

  0