Design comparison
Solution retrospective
- What do you think about semantics in my html?
- What do you think about my CSS? Do I approach this correctly?
Community feedback
- @DreamleafPosted 11 months ago
Just a small thing, but in the QR code alt text, you could be more descriptive and say where the user would be taken. So like, alt="QR code to frontendmentor.io".
When a screen reader uses the alt text it's helpful for the user to know the function of the image. If an image is purely decorative, then leave alt text blank, this will be skipped, but be as helpful las possible for 'functional' images.
Hope this helps!
Marked as helpful1@Robert-Lukasz-KsiazekPosted 11 months ago@Dreamleaf Thank you so much for your valuable input.
1 - @danielmrz-devPosted 11 months ago
Hello Robert!
Your project looks great!
About your questions:
What do you think about semantics in my html?
- Your HTML is good. I'd just use the tag
main
instead of the tagsection
in this case. We just have one element on the screen, and since it's the main element, it makes more sense to usemain
in this case.
What do you think about my CSS? Do I approach this correctly?
- Very good CSS structure too. I just suggest you to use variables. Especially for
colors
andfonts
. I know this project is small, so you don't have to write many lines of code, but it's because of this same reason I think it's a good opportunity to practice their use.
I hope it helps!
Other than those little details, you did an excelent job!
Marked as helpful1@Robert-Lukasz-KsiazekPosted 11 months ago@danielmrz-dev thank you for your time and expertise.
1 - Your HTML is good. I'd just use the tag
Please log in to post a comment
Log in with GitHubJoin 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