front mentor huddle landin page solution

Community feedback

• 

  Raymart Pamplona•16,100

  @pikapikamartPosted over 3 years ago

  Great work on you first challenge here in FEM.

  The layout is good in desktop and mobile view is good.

  Suggestions would be that

  -. Instead of adding height: 100vh in your body tag, you can specify only min-height: 100vh. This way, it will make sure that your website will have a minimum height, based on the viewport, or the screen of the user.

  • Also your footer social media icons, it will be awesome if you put those element inside an a tag. Because those are links right, so nesting them or putting them inside an a tag will be really good.

  Just keep doing these challenges and also reading or studying more about html, elements and it will really help you. Also freecodecamp.org is a great website to study a lot in different fields. Good job ^^

  0
• 

  Alexei•831

  @Batareika007Posted over 3 years ago

  Hi Alex! I think for beginner you did a very nice job. the webpage responsive, only need to make some cosmetic with positions. It's look great on mobile. More practices and I would like to see more of your future works, well done !

  0