Submitted almost 3 years ago
Responsive landing page using HTML, CSS, and bootstrap
@danielasakpa
Design comparison
SolutionDesign
Community feedback
- @RioCantrePosted almost 3 years ago
Hello there! Nice work with this one. Viewing your solution, I would suggest the following for you...
- Adjust the top margin into
margin-top: 9em;
in the.Main-div
rule set - Remove this part if it is not included in the design...
<h1 id="qr-code">QR-code</h1> <p>QR code component solution</p>
- Wrap
container mx-auto Main-div
with specific tag likemain
andattribution
withfooter
tag for readability - Include description to
alt
inimg
tag and should not be empty
Overall you did good and Keep up the good work!
0@RioCantrePosted almost 3 years ago@danielasakpa No worries! I would appreciate it if you mark it as helpful. Cheers!
0 - Adjust the top margin into
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