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

newbie card with css

MEHDI204β€’ 90

@MEHDI204

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

Woldu Tsegazgiβ€’ 910

@weldu0

Posted

πŸ‘‹ Hey, @MEHDI204! I have a few suggestions for your project:

1. You don't need to use position for centering. Since you already have flexbox set up correctly, just add min-height: 100vh; to the body, and everything will be perfectly centered. It should look like this:

body {
/* Your other code */
min-height: 100vh;
}

2. Since you’re already using flexbox, you can remove any position properties and use max-width instead of width. This will make your solution more responsive.

Hope you find this helpful! 😊

Marked as helpful

0
ortiz-antonioβ€’ 120

@ortiz-antonio

Posted

Great start! Could you explain why you're using position: absolute? The text seems to shift from the card when I resize the window.

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