Design comparison
SolutionDesign
Solution retrospective
Please feel free to review my code.
Community feedback
- @dewslysePosted over 3 years ago
Hello Nabil!! Nice work on your submission. Page is responsive across devices. Some things to note:
- Your page is missing the background image.
- You could wrap the main content in your html in a
<main>
. This would help with some of the accessibility issues<section class="card">
could be<main class="card">
- The Proceed to Payment button could use a :hover state
- Your could add
aria-hidden: true
to your hero image for improved accessibility since it's decorative
Happy coding
Marked as helpful1@Nabil19911Posted over 3 years ago@dewslyse Thank You for your feedback. It is helping me to improve my skill and feel free to check again I have improved my style a bit.
1
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