Design comparison
Reports
Accessibility report (3)
warning
All page content should be contained by landmarks
<div class="attribution">
Challenge by
<a href="https://www.frontendmentor.io?ref=challenge" target="_blank">Frontend Mentor</a>. Coded by <a href="#">Qazi Aimal</a>.
</div>
Learn more Qazi Aimalβs questions for the community
I hope you guys like my work...
Community feedback
Kacper Kwintaβ’ 855
@kacperkwinta
Posted
Good job Qazi ππ
Keep up work!
Some tips to make your solution even better:
- to wrap all
<div>
elements and other important stuff, it's good practice to use semantic meaning elements like<section>
,<article>
,<main>
,<footer>
etc... - Proceed to Payment btn should have white font color
There are also other not much important differences, look closer at design and try your best to make your solution similar β
Marked as helpful
0
Qazi Aimalβ’ 150
@aimal-qazi
Posted
@kacperkwinta Thanks next time i will use semantic elements and thanks for the feedback
1
Please log in to post a comment
Log in with GitHubJoin our Slack community
Join over 180,000 people taking the challenges, talking about their code, helping each other, and chatting about all things front-end!