Submitted 6 months ago

Web page created using mobile-first approach and HTML and CSS.

Oliver• 50

@Liv-er

Preview site(opens in new window)View code(opens in new window)

This is a solution for...

QR code component

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

I am proud of my code because it is readable, and anyone could understand it. What I would do differently next time is to use JavaScript to make the page more dynamic.

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

The challenges I faced creating this initial solution were that it was my first time using Figma and my first professional project where I was given the design and images to use, and I just had to create the page.

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

In terms of my code, I would like:

Best practices
Improved readability.

Community feedback

Daniel 🛸• 44,230

@danielmrz-dev

Posted 6 months ago

Hello, @Liv-er!

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!

Please log in to post a comment

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