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 Component

Ekta Chaudhary• 10

@itsekta

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

j124klo• 40

@j124klo

Posted

First and most obvious, thats not the same QR code you were given, and even if it's just Rickroll I wouldn't risk by scaning it. Remember that unknown QR and links found on internet can be really dangerous. By just scanning some random QR you can give out your personal information.

That aside, about accual code (disclaimer - i'm no expert in JS nor react so i will address what I know about):

  • Your page is not phone viable. As you can check in dev tools (by pressing F12), when you select phone view option, than select change orientation to vertical, your backgroud color doesn't cover the whole screan(*). Also the top of the picture gets cut off out of screan(**).

(*) It might not be enough, but I recomend setting background color as body property instead of your conteiner property, example:

body { background-color: #d5e1ef; }

(**) If you set your middle-parts' margin to auto, it should include the whole thing even in the phone mode, example:

#Main-div { background-color: white; margin: auto; <-----------(the fix) text-align: center; width: min-content; height: min-content; border-radius: 1rem; padding: 0.4rem; }

  • You aren't using the correct font, though you have a lot other. Just in case, the font is called "Outfit". You can just add this:
<link href="https://fonts.googleapis.com/css2?family=Outfit:[email protected]&display=swap" rel="stylesheet" />

Other than that, and small things (size of the thing, border-radius of the white thing, text color, QR being black and gray in dark mode (how? why?)), it looks good enough.

0
P
Steven Stroud• 11,810

@Stroudy

Posted

Hey, You repository is private I believe we cant see it to give you feedback.

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