html, css and flex.

Community feedback

• 

  Will Cook•150

  @willcook4Posted almost 5 years ago

  Great first crack... I think your 'per month' text and your 'why us' list is the wrong color. Check the design again, I don't have the .sketch file though.

  Consider cleaning up the index.html to make it flow. The indentation is a bit wonky making it harder to read.

  'Why us' should probably be an unordered list. It reads like a list.

  Consider cleaning up the spacing in the stylesheet. Your line breaks are inconsistent making it harder to read. Your indentation is inconsistent for .right-box on mobile.

  Give it a tidy up, in general it's looking the part

  3

  ---

  

  Brian Patterson•130

  @amidabrian51Posted almost 5 years ago

  @willcook4 Thanks for your suggestions. Will try and do these soon.

  0
• 

  argel omnes•1,800

  @argelomnesPosted almost 5 years ago

  Hi Brian,

  Good job! Adding to @willcook4 :

  • Link your html to the correct font. CSS is already using 'Karla' so that's good.
  • You can remove the horizontal paddings in headings and paragraphs and instead use it in their containers. ie, .left-box and .right-box

  1

  ---

  

  Brian Patterson•130

  @amidabrian51Posted almost 5 years ago

  @argelomnes Thanks for your help. Will try and do these soon.

  0