Order Summary Component with a little bit of SASS :)
Design comparison
Solution retrospective
This was quite fun i gotta say . at last i got the opportunity to be comfortable with css grid
i also for the first time used SASS which i am not sure why i was not using it till now. it is really Awesome :)
as for questions i dont have specific questions this time but i do want some tipps on how to better organize scss code
Community feedback
- @NaveenGumastePosted over 2 years ago
Hay ! selehadin Good Job on challenge
These below mentioned tricks will help you remove any Accessibility Issues
-> Add Main tag after body
<main class="container"></main>
-> Learn more on accessibility issues
If this comment helps you then pls mark it as helpful!
Have a good day and keep coding 👍!
Marked as helpful1 - @RioCantrePosted over 2 years ago
Hello there! Nice work in completing this challenge. Viewing at your solution, I would suggest the following for you...
- Add
background-color: hsla(234, 95%, 95%, 0.79);
andbackground-position: top;
in thebody
rule set. Removebackground-attachment: fixed;
, adjustbackground-size: 100%;
. - Add
cursor:pointer
for thebutton
- Wrap
attribution
with specific tag likefooter
for readability. To know more, refer it with Semantics - Include description with the
alt
inimg
tag - Remove
position: absolute;
,bottom: 10px;
andright: 0;
inattribution
. Addpadding: 1rem
Overall you did great and Keep up the good work!
Marked as helpful1 - Add
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