HTML5, CSS3, Flexbox

Community feedback

• 

  P

  tediko•6,680

  @tedikoPosted over 3 years ago

  Hello, Siddhesh Kamble! 👋

  Well done on this challenge! In addition to what palgramming pointed here's a few things I'd suggest you to take a look at:

  • Change the alt attributes for the .card icons, as they don't add any extra context for screen reader users. Since your images are decorative your alt text should be provided empty (alt="") so that they can be ignored by assistive technologies.

  Keep up the good work! 💪

  0

  ---

  

  Siddhesh Kamble•330

  @Sk7867Posted over 3 years ago

  @tediko Thank you, will do.

  0
• 

  P

  Patrick•14,265

  @palgrammingPosted over 3 years ago

  Look at your page at a browser width of 1081px and you will see you text on one button spanning over two lines and the other buttons are one line. I personally would just a fixed width and height on the buttons and not padding dictate their size. Or put a min-width on them to keep them from growing too small

  0

  ---

  

  Siddhesh Kamble•330

  @Sk7867Posted over 3 years ago

  @palgramming Thank you for pointing that out and the suggestion as well. I will keep this in mind.

  0