Completed Using Atom (HTML, CSS and Bootstrap)

Community feedback

• 

  Alex•1,090

  @AlexBueckigPosted almost 5 years ago

  Hi Vaibhav,

  one thing I'd advise you is to have a look at HTML5 semantic elements. They greatly improve accessibility (for both search engines and screen readers) and make your markup easier to understand for everyone. ;)

  3

  ---

  

  Vaibhav Chandak•200

  @Vaibhav-chandakPosted almost 5 years ago

  @AlexBueckig Ok Thank you.

  0

  

  Ritesh Dewoolkar•0

  @Ritesh1432Posted almost 4 years ago

  @AlexBueckig They also help the search engines to retrieve information faster

  0
• 

  argel omnes•1,800

  @argelomnesPosted almost 5 years ago

  Hey Vaibhav,

  That's fine. You'll improve as time goes by if you keep coding. I suggest moving your scripts to the bottom, before </body>. This is to improve page load time. Sticking to min-width only is less of a headache for me. Give it a try on your next challenge.

  Good job!

  2

  ---

  

  Vaibhav Chandak•200

  @Vaibhav-chandakPosted almost 5 years ago

  @argelomnes Ok Thank you for the suggestion. I didn't understood your min-width point.

  0

  

  T

  Matt Studdert•13,611

  @mattstuddertPosted almost 5 years ago

  @Vaibhav121121 Argel is suggesting you try using min-width media queries instead of max-width. They're great, as you typically end up writing less code and they also have the benefit of loading in fewer styles for your mobile users.

  Here's a great run-through of responsive on Interneting is Hard, which goes into this mobile-first approach if you're interested.

  1