Mobile First site using HTML BEM and CSS

Community feedback

• 

  Arya Anish•1,492

  @AryaAnish121Posted almost 4 years ago

  Hi David Ouma,

  I have two suggestions:-

  • 1. you can give the landing page a padding
  • 2. and same for footer

  And all is 100% perfect

  please upvote this comment

  2

  ---

  

  David Ouma•270

  @daudi13Posted almost 4 years ago

  @AryaAnish121 thanks for taking your time. that was very helpful. just a quick question, how do you write the properties in the comment section so that they appear as they do, like how (like how the word padding looks different from other words in your comment above), please advice.

  0
• 

  Al Nahian•775

  @alnahian2003Posted almost 4 years ago

  Hey David Ouma

  Your project looks fine and nice, and it's really good to see that you've used HTML BEM but you could do something more to make the project more accurate.

  At first,

  • In the intro/hero section, you could imagine the entire section like a single row with two different columns, and the coulmns width will be 50%-50% of the width of the container that's containing the columns.
  • Make the intro image/hero image a little bit larger as same as the design provided in the starter file.

  That's It!

  Happy Coding :)

  Don't forget to UpVote any comment that you've found helpful in frontendmentor.io

  2

  ---

  

  David Ouma•270

  @daudi13Posted almost 4 years ago

  @alnahian2003 thank you very much that was very helpful.

  1

  

  Al Nahian•775

  @alnahian2003Posted almost 4 years ago

  Hey David

  It's my pleasure, mate!

  Happy Coding

  0