TIP CALCULATOR

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Nice work with this one. Looking at your solution, I would suggest the following for you...

  • Add media queries for the responsive design. With max-width:1066px as the starting breakpoints. Use Flexbox for the alignment
  • Add cursor: pointer for the hover state
  • Adjust the background color of the .reset-btn into background: hsla(183, 78%, 24%, 1); and hover state with background: hsla(172, 61%, 77%, 1);
  • Wrap the whole content container with specific tag like main. For HTML structures, refer it with this one Semantics
  • Remove commented code to keep it clean. Alternative is to uncomment and wrap the attribution with footer tag. Add name and links with your work/portfolio

  Overall, you did good.Hope this helps and Keep it going!

  Marked as helpful

  0