Skip to content
Submitted almost 2 years ago

QR code design for everyday use

Md Alqma Ansari•40
@Md-Alqma
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?

While completing this challenge, I learned about some new CSS properties. I will try to complete using less CSS lines.

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

Responsiveness was a bit challenging. I used rem and em for CSS.

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

Colors and font selections.

Community feedback

  • Hassia Issah•50,650
    @HassiaiPosted almost 2 years ago

    Replace <h3> with <h1> to fix the accessibility issues.

    To center .container on the page using flexbox, replace the height in the body with min-height: 100vh.

    To center .container on the page using flexbox:
body{
min-height: 100vh;
display: flex;
align-items: center;
justify-content: center;
}

    There is no need to style .image-container. give the img a max-width of 100% instead of a width and height value.

    Give h1 the same font-size as p.

    Use relative units like rem or em as unit for the padding, margin, width values and preferably rem for the font-size values, instead of using px which is an absolute unit. For more on CSS units Click here

    Hope am helpful.

    Well done for completing this challenge. HAPPY CODING

    Marked as helpful
    0
    Md Alqma Ansari•40
    @Md-AlqmaPosted almost 2 years ago

    @Hassiai Thank you so much for your feedback. Looking forward to becoming a good developer with help of developers like you...

    0

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