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

css breakpoint, flex display, css var for colors

@elekviktor32

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

P

@ownedbyanonymous

Posted

Hie Elekviktor 👋🏾

Great work on the QR Code Challenge 🔥🔥 I will be giving feedback on your work using the following guidelines

  1. Does the solution include semantic HTML? NO
  2. Is it accessible, and what improvements could be made? NO (read notes for more details)
  3. Does the layout look good on a range of screen sizes? YES
  4. Is the code well-structured, readable, and reusable? YES
  5. Does the solution differ considerably from the design? NO

Notes

  • The code works and your solution is very close to what was provided in the Figma designs (Well done!). However, l think we could leverage the semantic HTML tags like <main>...</main> instead of the <div class='container'>...</div> to indicate that this is the main focus or content of your page.
  • You could also wrap the bottom content inside a footer tag <footer>...</footer> instead of the <div class='attribution'>...</div>.
  • Semantic tags clearly convey the purpose of each section.
  • Additionally, screen readers used by visually impaired people rely on semantic HTML to understand the structure of a webpage and navigate it effectively. Screen readers can interpret headings, recognize landmarks like headers and footers, and present the content in a way that makes sense to the user. This provides accessibility of your webpage.
0

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