Skip to content
Submitted 7 months ago

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

Oliverā€¢50
@Liv-er
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...QR code component
  • HTML
  • CSS
1newbie

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,250
    @danielmrz-devPosted 7 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!

    1

Please log in to post a comment

Log in with GitHub

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