Design comparison
Solution retrospective
I am looking forward to any suggestions on this page.
Community feedback
- @VCaramesPosted about 2 years ago
Hey there! 👋 Here are some suggestions to help improve your code:
-
The Alt Tag description for the QR image needs to be improved upon. Its needs to tell screen reader users what it is and where it will take them to when they scan it.
-
Change
width
tomax-width
in your component’s container to make it responsive. You will also want to remove theheight
as it is unnecessary. -
Change
width
tomax-width: 100%
in your image to make it responsive. You will also want to remove theheight
as it is unnecessary. -
A media query is not needed for this challenge. Using responsive properties will do in making your content responsive.
If you have any questions or need further clarification, let me know.
Happy Coding! 👻🎃
Marked as helpful0@SYEDUMAIR007Posted about 2 years ago@vcarames Thank you for your valuable feedback, and yes, I will do as you said.
0 -
- @MelvinAguilarPosted about 2 years ago
Hi @SYEDUMAIR007 👋, good job on completing this challenge, and welcome to the Frontend Mentor Community! 🎉
I have some suggestions you might consider to improve your code:
- Use the
<main>
tag to wrap all the main content in your solution rather than<container class="main">
. - In HTML there is no
<container>
tag. - Use
<footer>
instead of<div class="attribution">
. The<footer>
element contains authorship information. - Instead of using pixels in font size, use relative units of measure like
rem
orem
. The font size in absolute length units (px) does not allow users with limited vision to change the text size in some browsers. Reference. - To make alternative texts more useful, add descriptive text to the alt attribute of the QR image to explain what the QR image does.
I hope those tips will help you! 👍
Good job, and happy coding! 😁
Marked as helpful0@SYEDUMAIR007Posted about 2 years ago@MelvinAguilar Thank you for your suggestion, i am going to improve my code right now
1@SYEDUMAIR007Posted about 2 years ago@SYEDUMAIR007 hey improved my code as per your suggestions, check out and comment
0 - Use the
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