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>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 GitHub
Join 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!