Responsive landing page using HTML5, CSS3, JavaScript, Flexbox

Community feedback

• 

  P

  Emmilie Estabillo•5,540

  @emestabilloPosted almost 4 years ago

  Hi @obioraigboanusi, congrats on this challenge! Here are some thoughts:

  • I think if you moved main right before the hero div, you might not need the sr-only header on the why section, since it is an exact copy of the text in the hero.

  • On tablet view, the paragraph under the why section is left-aligned.

  • Also on tablet, the dark overlay for the menu is not covering the entire width of the viewport. Width is at 100%, which means it is 100% of the parent nav-wrapper, which also means if the parent has margins (it's set to auto), the overlay will follow this setting as well.

  • The articles are missing their anchor tags that link to the full story

  • Might be better to keep the outline on the buttons so users can get cues when they're tabbing through the site

  Hope this helps :-)

  1

  ---

  

  Obiora•0

  @Obiora-PrimeTechPosted almost 4 years ago

  @emestabillo Thanks for your detailed review. I will effect the adjustments. Thanks a lot.

  0