Responsive Landing Page Using Bootstrap

Community feedback

• 

  Eric Salvi•1,350

  @ericsalvi

  Posted about 4 years ago

  Hey Harshit,

  Looks pretty good for your first challenge solution. Only spotted 2 things when it comes to the design itself. The footer seems out of place with the white background. Not sure if you can play with that so it doesn't look like a mistake but rather part of your design? If you are not 100% happy with it, it is not required so you could just remove it altogether. I have done that occasionally.

  I think the image on the desktop would work much better as an image set in the background through CSS rather than being an image element. This way it flows much better as the screen gets smaller.

  For the code, I'd try to avoid inline styles as much as you can. Also, as someone on the outside looking at the code and if I were tasked to fix it, I would be confused as the class names are not specific at all to know what they are for. To learn a standard way of naming your content, check out the BEM naming convention.

  Keep up the momentum!

  01