Huddle landing page with single introductory section

Community feedback

• 

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted almost 2 years ago

  HI! Your solution looks nice though there are a couple of things you can improve which I hope will be helpful! 😊

  In the social media div, you need to place the icons inside a tag, because the purpose of a tag is to redirection to other social media platforms, in this case, it's not just decoration.

  <div class="footer-icons">
  			<i class="fa-brands fa-facebook-f"></i>
  			<i class="fa-brands fa-twitter"></i>
  			<i class="fa-brands fa-instagram"></i>
  		</div>
  

  I would also add some transitions for active states (when colors change on hover). It creates more interactivity and makes the project look cooler. Active states can be done on buttons, links, titles which act like links, or anything else, you choose.

  You can read more about it here, in case you haven’t done much of it:
https://www.w3schools.com/css/css3_transitions.asp

  Also, a quick question, why did you need class for h1, for example? Because you have only one h1 on the page so you don't need any class to target this tag for CSS, for example.

  Marked as helpful

  1

  ---

  

  Elif Seyhan•180

  @elifseyhanPosted almost 2 years ago

  @catherineisonline Hi! Thank you for useful suggestions. :) I made some changes on my codes. I think it looks better thanks to you.

  Happy coding 🥰

  0

  

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted almost 2 years ago

  @elifseyhan Nice! Glad if that helped 😊

  1