Design comparison
SolutionDesign
Solution retrospective
Hello everyone, It's been long time that i don't code I forget a lot of time now i need to pratice everyday for remenber a lot things any help would be helpful don't insist on correcting me Thank you !!!!!
Community feedback
- @vanzasetiaPosted almost 2 years ago
Hi, @steeven509! π Welcome back to coding! π
I have some suggestions to improve this solution.
- Remove
<div class="qr-block-img">
. Make the<img>
as a block element and set the styling to the image directly instead of using<div>
. - Improve the alternative text of the QR code by describing the purpose of the QR code. "QR code to frontendmentor.io" can be a good alternative text for it.
- Use a CSS reset whenever you start a new project. This can help you set the styling foundation easily. My recommendation β A Modern CSS Reset
- Remove
height: auto;
andoverflow: hidden;
from the.qr-container
styling. It is because:- For the height, by default, the height of the element is set to
auto
and - For the
overflow
, there is no issue whether the property is there or not.
- For the height, by default, the height of the element is set to
- Never limit the height of the
<body>
. It will not allow the users to scroll the page when the page content needs more height. Try to look at the site on a mobile landscape view to see the issue. So, usemin-height
instead. - Never use
px
unit for font sizes. Userem
orem
instead. Relative units such asrem
andem
can adapt when the users change the browser's font size setting.
I hope you find this useful. π
Marked as helpful1@steeven509Posted almost 2 years ago@vanzasetia itβs very helpful my friend how can i keep in touch with you on other social networks
0@vanzasetiaPosted almost 2 years ago@steeven509
I am available on the following social media platforms:
- Vanza Setia (@vanzasetia) / Twitter: I often share what I've been working on or resources.
- Vanza Setia | LinkedIn: I usually chat with people here. So, if you want to message me then this is the best place. I also share what I've been working on here.
P.S. I notice that we have connected on LinkedIn. π
Marked as helpful1 - Remove
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