Mobile first order summary component using Sass
Design comparison
Solution retrospective
Hello everyone! Any feedback on my solution will be appreciated!
Community feedback
- Account deleted
Hi there ๐
Congratulate on finishing your project ๐. You did a great job ๐ก.
I give some suggestions to help you take your project design to the next level ๐๐
- One thing left there is the color of the box-shadow in the cards should not be black, change it to dark blue or as in the button
- Also add
cursor: pointer
to the button
Happy coding โ
Maqsud
Marked as helpful0 - @RioCantrePosted over 2 years ago
Hello there! Good job in this challenge. Viewing the solution, I think you should consider the following as wellโฆ
- Remove unnecessary code to keep it clean
- Add the background image of the design. Include it inside the
body
rule set - Add
cursor: pointer
for the mainbutton
Above all, Well done! Keep it up and Hope this is helpful!
Marked as helpful0 - @denieldenPosted over 2 years ago
Hi Ruan, I took some time to look at your solution and you did a great job!
Also I have some tips for improving your code:
- try to add a little
transition
on the element with hover effect - For add the top image in the background just put more specific background properties to the body:
background: url("../img/pattern-background-desktop.svg") no-repeat top center; background-size: contain; background-color: #e0e8ff;
Overall you did well :)
Hope this help and happy coding!
Marked as helpful0 - try to add a little
- @RuanEsterhuysePosted over 2 years ago
Thanks for the feedback! I appreciate it.
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