Skip to content
Submitted 6 months ago

Simple QR Code

ijohnst•230
@ijohnst
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?

Remembering how to code I took a long break from it

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

Just remembering syntax

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

None right now, this was more to get used to coding again

Community feedback

  • Øystein Håberg•10,710
    @Islandstone89Posted 6 months ago

    Hi, great job!

    Some suggestions:

    HTML:

    • The alt text must also say where it leads(frontendmentor website). A good alt text would be "QR code leading to the Frontend Mentor website."

    • I would change the heading to a <h2> - a webpage can only have one <h1>, which is reserved for the main heading. As this is a card heading, it would likely not be the page's main heading.

    CSS:

    • I like to add 1rem of padding on the body, to ensure the card doesn't touch the edges on small screens.

    • It is common to add display: block and max-width: 100% on images.

    • As the design doesn't change, there is no need for any media queries. When you do need them, they should be in rem or em, not px.

    Marked as helpful
    1
  • Jennifer•70
    @jeniegPosted 6 months ago

    Looks good on a variety of screen sizes.

    Marked as helpful
    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