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

QR component using Grid and flexbox

Chris Jay 220

@chrisjay358

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


This was a fun one. What do you think I could've done better? Or better still how would you have approached it? Thank you as always!

Community feedback

@rostyslav-nazarenko

Posted

Hi! Great solution!

Almost nothing to add except just a few things

  • alt should describe an image or its function and only be empty if an image is for decoration. In this challenge alt should be QR code to frontendmentor.io
  • I used the font hack too, but Melvin Aguilar provided feedback that it's better not to use this. Here is the article
  • I'm not sure about this, but I wouldn't use flexbox that much. Why? I have no idea. 😁 But feedback on my feedback would be appreciated

Have a nice day!

Marked as helpful

0

Chris Jay 220

@chrisjay358

Posted

@rostyslav-nazarenko Thank you for taking the time to give some feedback.

  • For the alt i almost forgot to put that, thank you for reminding me, I've corrected that.
  • For the font hack I also saw something in Kevin Powell's YouTube channel that right now everything is turning to base 16. You know the 10 makes it easy to do calculations, some of us suck at math and are lazy to open calculators. Might take a hard time going back to base 16. I would read the article.

Thank you, do make to drop more feedbacks*

1

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