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 code challenge with css flexbox

@Mohammed-Abol-Fotouh

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 is my solution for the QR code challenge please review my solution and tell me if there is best practice should i follow thanks in advance

Community feedback

@ShannTimothy22

Posted

Congratulation on finishing the challenge 🥳 , your solution looks great and there's no accessibility issues and html issues, I have some few suggestion

• The container seems to have fixed height, it makes your content overflowing you can fix that by removing the fixed height

• It also seems there’s a few problem with some of the breakpoints (responsiveness), you can fix the problem by simply adding more breakpoint

• For the padding and margin, it’s recommended to use scaling such as rem

If there’s still any problem I recommended you watch Kevin Powell on youtube, the channel will help you a lot on your coding journey, other than that good job on finishing the challenge and happy coding 😁!!

Marked as helpful

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