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

Responsive Card Component

polukarp 190

@polukarp

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


What do you think? Any critic would be good. I think my image sizing is off, so if you have any idea how to implement that better, I'm all ears.

Community feedback

Vanza Setia 27,795

@vanzasetia

Posted

Hi! 👋

Congratulations on completing this challenge! 🎉

One suggestion I have is to remove the role="contentinfo" from the div. What is the purpose of doing this? I would not recommend using ARIA attributes unless you have to.

I have three recommended videos. The first one tells how hard HTML is (HTML is not easy). The other two talk about modern CSS techniques and approaches.

I hope this helps! Happy coding!

Marked as helpful

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