Responsive Landing Page

Community feedback

• 

  P

  Theo Harris•360

  @Theosaurus-RexPosted about 2 months ago

  Hey, this looks pretty close to the design, well done!

  Some notes on design diffs:

  • The background colour is meant to be a slightly off-white color, which is provided in the style-guide doc in the starter code
  • Your header images look a little distorted in the screenshots but are okay on the live site for me, but you might want to double check them just in case!

  In terms of the code:

  • I don't really think a nav element is appropriate where you've used it, as there are no navigation links inside there, just a logo
  • Your heading elements need to be in order from smallest to highest number, and you can only have one h1 element per page
  • Your buttons should appear interactive when moused over, but my cursor remains normal when mousing over them - I can't see anything in the code that would override the default behaviour though, so I'm a little confused as to why I'm seeing this behaviour

  0