I used flexbox to arrange the various components

Community feedback

• 

  T

  Grace•31,270

  @grace-snowPosted almost 4 years ago

  Hey,

  Well done on this, some really nice work. And very impressive for 1 month into your coding!! 👍 👍

  There's a few things already covered in your report - try to fix those.

  If you want to add some polish and make yours match the design more closely

  • add the bg image to the page with css
  • add shadows to your buttons like in the design (I think their shadows use a hue from the button color for the shadow?)
  • add focus and hover states to buttons

  I'd also use the html section element to wrap some of those sections of content and a header for the header

  With the footer, I would use flexbox too for the overall footer layout. ☺ You can probably keep the main list of links in one <ul> though, and use column properties in css to split it into the 3 columns at desktop size.

  Good work, keep learning, keep coding

  0

  ---

  

  Iyanu Elijah Egbuwalo•145

  @enigmirePosted almost 4 years ago

  Thank you Grace. I really appreciate your input. I will effect the changes and move to the next challenge ☺️

  1