easybanking-frontend-mentor using tailwind and react

Community feedback

• 

  

  Emmilie Estabillo•5,580

  @emestabillo

  Posted almost 3 years ago

  Hey @abhikr4545, looks good, congrats! Solving the issue with the background would involve removing overflow: hidden to the parent section, and then adding a higher z-index to the nav as follows:

  position: relative;
      z-index: 1;
      background-color: white;
  

  Just a couple more thoughts:

  • I suggest increasing the breakpoint because the layout looks tight at 768px. The hero paragraph is touching the phone image and the side by side columns have little breathing room. The text on the articles are also getting cut off and the images are stretched.
  • articles, footer list items, and social icons could be wrapped in <a> tags

  Hope this helps!

  10

  Marked as helpful