Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

qr code challenge

Finneyβ€’ 3,030

@Finney06

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


All feedback and and correction is welcomed

Community feedback

Vanza Setiaβ€’ 27,855

@vanzasetia

Posted

Greetings, Finney! πŸ‘‹

Great work on this challenge! Your solution looks pretty good! πŸ˜€

Grace Snow has given some incredible feedback about accessibility. πŸ™Œ Here is some feedback from me.

  • I would recommend trying to simplify the HTML markup. Currently, there are too many div to create the card. I suggest using the body element as the flex container to align the card to the center of the page and use the main element for the card element.
  • I highly suggest finishing this challenge without any media queries. Both the desktop and the mobile layout have the same spacing, font size, etc. So, it's possible to finish this challenge without a single media query. πŸ˜‰

That's it! Happy coding! 😁

Marked as helpful

1
P
Graceβ€’ 27,950

@grace-snow

Posted

Hi

Well done on this. Just a few small improvements needed

  • Indent your code consistently. your code editor can even do this formatting automatically for you
  • you need a main element and a footer element (for attribution) on this. These are called landmarks and will solve the warnings in your report
  • remove the brs from the paragraph. They make the lines break in strange places on my phone screen and would be announced as "break" by screenreaders. Don't try to ever control line breaks like this. Let the width of the container determine where text needs to flow onto the next line - the browser can handle it fine

Marked as helpful

1

Finneyβ€’ 3,030

@Finney06

Posted

@grace-snow thanks for the feedback it's really helpful but I still don't understand the second feedback well (about main and footer element).

still kinda a newbie.πŸ˜–

0
P
Graceβ€’ 27,950

@grace-snow

Posted

@Finney06 landmarks are structural elements in a page, children of the body. Main, footer and header are the most common landmark elements you'll need

Marked as helpful

1

Please log in to post a comment

Log in with GitHub
Discord logo

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