Design comparison
Community feedback
- @VCaramesPosted about 2 years ago
Hey @kylejrhoden, some suggestions to improve you code:
-
The “Illustration” and Music Icon serves no other purpose than to be decorative; It adds no value. The Alt Tag should left blank and have an aria-hidden=“true” to hides it from assistive technology.
-
Remove this line of code, "<div class="bg__image"></div>" and implement the background image to the Body Element.
-
There is only one heading this challenge and that is the ”Order Summary".
-
Your "button" was created with the incorrect element. When the user clicks on the button they should directed to a different part of you site. The Anchor Tag will achieve this.
-
You want to work on the styling in the mobile view. Everything begins to get pushed in and the card stretches vertically.
Happy Coding! 👻🎃
0 -
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