Mobile First Easy-Bank Landing page

Community feedback

• 

  P

  tediko•6,680

  @tedikoPosted over 3 years ago

  Hello, Aman Patil! 👋

  Good effort on this challenge! 🎉 Everything works good and responds well. Take a look at:

  • Add :focus pseudo class to interactive elements like anchors, buttons etc. Use outline property to make your website more accessible to keyboard users. Focusable elements like anchor, buttons or inputs they have applied default :focus pseudo class with outline property. These default styles are subtle and hardly visible tho. Furthermore every browser has a slightly different default style for the outline, so you probably want to change the default style. Read more about why we should change focus styles.
  • You can add class to your html elements likesection footer etc instead of creating another unnecessary div within that element to style it.

  Good luck with that, have fun coding! 💪

  1

  ---

  

  Aman Patil•885

  @Amanpatil-DevPosted over 3 years ago

  @tediko thank you so much for your kind feedback, I will refactor as soon as I can and I will keep it in mind

  0
• Account deleted

  Nice one on completing the challenge.

  The 'request invite' button on the navigation bar looks like it needs a little fix, the other one looks fine, how about getting it to be like the other one. I like the border when you click on them.

  1

  ---

  

  Aman Patil•885

  @Amanpatil-DevPosted over 3 years ago

  @thulanigamtee thank you so much for your feedback I will refactor it as u said

  0