Order summary component

Solution retrospective
I would appreciate any feedback, thank you in advance.
Please log in to post a comment
Log in with GitHubCommunity feedback
- @pikapikamart
Hey, great work on this one. The wave background seems to be missing on this. Though the layout looks good.
Some suggestions would be:
- On the
body
tag, you declarationbackground-image: url(/images/pattern-background-desktop.svg);
won't work. Using/
root route in github works differently on our local machine. So, always make you specify the exact path of the file you are using, without using the/
root path. On your case, you should usebackground-image: url(./images/pattern-background-desktop.svg);
- The music icon should be left empty like
alt=""
. If an image is just decoration, better using emptyalt
on them. - Lastly, the
proceed to payment
andcancel order
should be insidea
tag and notbutton
. If this would be aform
button
can be used, but since it is not, use linka
on them.
Aside from those, great job.
Marked as helpful - On the
- @AmodeusR
Read the archives from Frontend Mentor and edit your readme.md archive, you didn't write it. You can follow the README-template they provide to know what you can write.
Marked as helpful - @VaNaChiMa
@pikamart Hi! How i can delete 2 of my solution, that are wrong, I published them by mistake?
- https://www.frontendmentor.io/solutions/coding-bootcamp-testimonials-slider-input-toggle-special-card-KkPZArc96
- https://www.frontendmentor.io/solutions/ping-single-column-html-css-flexbox-responsive-js-2SDTpNcI6
Join 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