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 using CSS

SAGAR-Cdyā€¢ 10

@SAGAR-Cdy

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 are you most proud of, and what would you do differently next time?

All feedback

What challenges did you encounter, and how did you overcome them?

maintaining the box model

What specific areas of your project would you like help with?

all

Community feedback

Daniel šŸ›øā€¢ 44,230

@danielmrz-dev

Posted

Hello there!

Congrats on completing the challenge! āœ…

Your project is looking fantastic!

I'd like to suggest a way to make it even better:

  • Using margin isn't always the most effective method for centering an element.

Here's a highly efficient approach to position an element at the center of the page both vertically and horizontally:

šŸ“Œ Apply this CSS to the body (avoid using position or margins in order to work correctly):

body {
    min-height: 100vh;
    display: flex; 
    justify-content: center;
    align-items: center;
}

I hope you find this helpful!

Keep up the excellent work!

Marked as helpful

0

SAGAR-Cdyā€¢ 10

@SAGAR-Cdy

Posted

@danielmrz-dev Thank you for helping me

1
Brian Hammerā€¢ 180

@BrianHammer

Posted

-The QR code does not center properly on my screen, and the gray border should only be added on large screens. I recommend removing the gray border, and add it last when everything else works -The text overflows from the card on my screen. This is because you have a fixed height set on the QR Card. Removing this will cause the height of the card to always fill all the elements inside -Add the 'text-align:center' style to your title like you did inside the <p> tag -The background colors do not match exactly. You can get the colors the designer used inside of the style-guide.md, and apply it to your design -The text color and font weight can be changed for the paragraph text. -A cool trick is that you can add draggable="false" inside your <img /> tag to prevent the user from dragging it EX: <img draggable="false" src="..." />

Overall, removing the fixed height of your card element will be the biggest improvement.

Marked as helpful

0

SAGAR-Cdyā€¢ 10

@SAGAR-Cdy

Posted

@BrianHammer Thank you for guiding me it mean a lot to me

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