Background image still tricky

Community feedback

• 

  Ahmed Faisal•5,095

  @afrusselPosted over 3 years ago

  Mobile view is good. Mobile nav is looking nice

  Marked as helpful

  2

  ---

  

  Renato Novaes•180

  @renatoalmeida49Posted over 3 years ago

  @afrussel Thank you so much, man!

  0
• 

  Mojtaba Mosavi•3,760

  @MojtabaMosaviPosted over 3 years ago

  Well done, take a look at the following things:

  1- Semantic markup it's all about choosing the right html element for a specific task, for instance header should be contained inside a h1 not a p tag. Achieving this is always not easy and even sometimes not possible that's when you use the generic element lik div. A example the item in the footer are expected to be link so it would make sense to place them inside a a tag.

  2- Add hover state to the social icons in the footer.

  Keep coding :=)

  1

  ---

  

  Renato Novaes•180

  @renatoalmeida49Posted over 3 years ago

  @MojtabaMosavi Thank you, mate! I will do this

  0