sass, flexbox

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted almost 5 years ago

  Nice work on this challenge, Ben! Your solution looks good. Here are a few pointers after taking a look at your code:

  • The curvy background pattern currently has background-attachment: fixed; on. This is something that could be updated to try and get it looking as close to the design as possible.
  • Also, as you can see by the design comparison your overall layout looks great. But the font sizes and vertical spacings are a little off. This is definitely worth having a go at refining.
  • You're using header, footer and main, which is great. But you could also add sections inside of the main content. For example, <div class="endorsements"> could be <section class="endorsements">.

  You've done a really good job. Let me know if you have any questions!

  0