Easybank landing page using React & Styled Components

Community feedback

• 

  Roman Filenko•3,335

  @rfilenkoPosted over 4 years ago

  Hey, great work, to avoid header links on hover - add transparent border-bottom, not you have it on hover as well, and definately use transition for buttons, links. Also work on those error in the report.

  Cheers, Roman

  0
• 

  Ramsay Sewell•585

  @ramsaysewellPosted over 4 years ago

  Hi Juan,

  This looks great, well done.

  I would be picky and say that the hover affects could be given a transition property to improve the hover effects.

  I noticed on your social icons in the footer, you have:

  .className svg path:hover { 
     fill: green;
  }
  

  However, the hover only occurs when you are hovering over part of the icon (that has a path). To slightly improve the UX, you can do this:

  .className svg:hover path {
     path: green;
  }
  

  Meaning that when you hover over the <svg> element, you want the path to change fill.

  Well done and happy coding!

  0