single-price-component grid

Community feedback

• 

  Vanza Setia•27,715

  @vanzasetiaPosted over 3 years ago

  👋 Hi Julian!

  👍 Good job on completing this challenge! Your website is looking good and responsive overall 🙌, so well done! I have some feedback on this solution:

  • Accessibility
    • The button doesn't have :hover style. Adding the :hover styling on the button will help the user notice that it is clickable.
    • Create a custom :focus-visible styling to any interactive elements (button, links, input, textarea). This will make the users can navigate this website using keyboard (Tab) easily.
    • I would recommend using ul and li instead of br tag for the list on the Why Us section. I have used Narrator on your solution and it just read everything at once, which it shouldn't.
  • Visual
    • I would recommend adding padding to the body element to prevent the card touching the edge of the screen. Right now on my mobile view (360px * 640px), the card having full height.

  That's it! Hopefully, this is helpful!

  Marked as helpful

  1

  ---

  

  Julian•280

  @Julr09Posted over 3 years ago

  @vanzasetia Definitely will keep that in mind, thank you so much for taking your time and reviewing my solution.

  0