Skip to content
Submitted 3 months ago

QR code component using HTML,CSS

MD.Shojeb Hossain Shojol•100
@shojebscodeplay
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?

Give me reviews

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

Easy one

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

CSS

Community feedback

  • Boris•2,850
    @mkborisPosted 3 months ago

    Great work, here are a few things to review

    • All content should be wrapped within landmarks. Wrap a main tag around the .container
    • Your image is missing an alt attribute like so alt="Qr Code to Frontendmentor.io"
    • It's best practice linking Google fonts directly in the HTML head section rather than directly in your CSS file as it enables asynchronous downloading, improving page load times.
    • Font-size should be written in rem not px
    0
    MD.Shojeb Hossain Shojol•100
    @shojebscodeplayPosted 3 months ago

    @mkboris thanks a lot for informatice review.

    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