Responsive easybank landing page using flexbox, css grid and sass.

Community feedback

• 

  P

  Joel•290

  @joelsalmeidaPosted over 3 years ago

  Hi Eric. Looks like you had a lot of work on this project. Congratulations. I have some tips to make everything even better.

  • You can use header, nav, main, footer, button... tags instead of just naming classes. You can put all sessions inside a main tag.

  • Don't forget the alt=" " tags in the images too.

  • In the original design the entire page follows a lateral alignment. Check the position of the header in relation to the rest of the body.

  • Check the font-size of your paragraphs. Some are too small.

  • Check the title color of articles on hover.

  • When opening the "mobile" hamburger menu the page should not be able to scroll. "I liked the animation, so smooth :)"

  Hope this helps. Keep coding.

  Marked as helpful

  3