sass, flexbox and grid.

Community feedback

• 

  Roc Tanweer•2,500

  @RocTanweerPosted over 3 years ago

  Hello 👋 @Chris

  Adding to what @pikamart said

  Your layout is quite far from the design...

  Use irfan view to measure the layout in the design image they provide or else have a premium membership, whatever you wish for

  Hope it helps ☺️ now you can have both functionality and design and not just the former one

  Happy coding 😉

  1

  ---

  

  Chris•740

  @chrisbailey85Posted over 3 years ago

  @RocTanweer thank you.

  0
• 

  Raymart Pamplona•16,100

  @pikapikamartPosted over 3 years ago

  Hey, great work on this one. The layout is good in both desktop and mobile.

  Everything seems resizes well when going in mobile as well.

  But I think that you are using the wrong font in your footer fag. Right now it is using times new roman which is default. Might want to look on that one.

  Also maybe add a padding in your header to the left and right side in mobile state. So that the brand logo won't be touching the sides of the viewport.

  Lastly, your facebook icon at the footer, it is different in terms of shape with its other sibling.

  Overall, you did a really good job^

  1

  ---

  

  Chris•740

  @chrisbailey85Posted over 3 years ago

  @pikamart thank you. I'll update that now.

  0