Order summary component using CSS custom utility classes
Design comparison
Solution retrospective
Hello everyone. This is my solution to the Order summary Component Challenge. I have tackled the problem by creating my own CSS utility classes and then using a few component to do slight changes. My questions are basically:
- Is my naming convention okay? Are there any improvements I can make?
- Is there anything I could have done better?
Awaiting replies and feedback, thank you for reading this and helping me out. i hope you have a good day!
Community feedback
- @denieldenPosted almost 3 years ago
Hi Farzeen, great job!
For fix the
svg
background add this property to the body:background-repeat: no-repeat; background-size: contain;
Hope this help :)
Marked as helpful1@FarzeenKistPosted almost 3 years ago@denielden Hey Deniel, Thank you for notifying and giving me out the answer for this. Have a good day/evening : )
1 - @alihuseynzade23Posted almost 3 years ago
Hey Farzeen! Well done.
--My only suggestion will be to add hover on the links and buttons. This will be great improvement in terms of smoothness.
Good luck :)
0@FarzeenKistPosted almost 3 years agoHey Ali, can you please clarify a bit more on what you mean? Are you talking about adding the hover effect to the button/link tags instead of using utility classes(I currently have 3 classes assigned to the links and buttons with hover and focus)? for e.g: .payment-btn:hover, .payment-btn:focus { cursor: pointer; background-color: hsla(var(--clr-primary-bright-blue) / 75%); border: 1px solid hsla(var(--clr-primary-bright-blue) / 75%); }
I'm thinking that I could also do a transition on each button/link to make it smoother.
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