Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

HTML5, CSS3, Flexbox

@Sk7867

Desktop design screenshot for the 3-column preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Are there any changes or improvements needed?

Community feedback

P
tediko 6,680

@tediko

Posted

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

@Sk7867

Posted

@tediko Thank you, will do.

0
P
Patrick 14,265

@palgramming

Posted

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

@Sk7867

Posted

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

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord