Submitted about 3 years ago
Order Summary Card - HTML, CSS, Flexbox - Desktop and Mobile
@chintriago
Design comparison
SolutionDesign
Solution retrospective
Any comments, criticisms, or advice is appreciated.
Community feedback
- @dewslysePosted about 3 years ago
Hello Chris! Congrats on your submission. Your component looks very good. Some things to note:
- Always remember to include landmarks (e.g.;
header, main, footer
etc) in your document. In this case, you could replace the.card-container
<div>
with a<main>
. Also, the.attribution
<div>
can be a<footer>
. - Replace the
background-image
onbody
withpattern-background-mobile.svg
at@media (max-width: 435px)
Happy coding!
1 - Always remember to include landmarks (e.g.;
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