Responsive landing page HTML, CSS

Community feedback

• 

  John Norman•880

  @norman02

  Posted about 5 years ago

  Thanks! I am pretty new to this. I will definitely use these suggestions

  00
• 

  Shashi Lo•1,345

  @shashilo

  Posted about 5 years ago

  Hey John. You did a great job using all the assets. It looks great, but it is missing some details. Here are the areas of improvement that I noticed:

  • You should create a .container with max-width: 1110px and used it within your sections.
  • Overall, paying attention to the detail of the design. For example, the hero section's title font size, call to action button's font size and alignment, etc. If you look at the design or overlay it onto your project, you'll see that you're close, but not pixel perfect to the design.
  • Instead of setting display: flex; to every main section, I'd use display: flex; on the parent and set flex-basis: % to create a width. You can also add align-items: center on the parent instead of each child element.
  • The box-shadow of the section's are too heavy.
  • Footer is missing the huddle image. Also, the content should start below the logo. Missing the social media icons as well. I know there isn't active states, but you should practice making all links real links.

  00