Design comparison
SolutionDesign
Solution retrospective
Hello, i relied a lot on @Hassiai's solution for this challenge after the gread advices i received on my 'Stat preview card component' challenge. Any advices to get better are welcome, thank you!
Community feedback
- @frank-itachiPosted over 1 year ago
Hello Laazabi Samir π. You did a good job!
I have some suggestions about your code that might interest you.
HTML π:
- You should wrap the testimonials section inside the
<main>
tag because that is also a very important part of the HTML. - The star images are used for decoration purpose so instead of omitting the alt attribute, you should add it and leave empty. Eg.
<img src="./images/icon-star.svg" alt=""/>
CSSπ¨:
- It is a little bit painful positioning the background images using the background property because you would need to use a lot of media queries. You could use the pseudo classes ::before and ::after to tackle it in a easier way.
I hope you find it useful! ππ Above all, the solution you submitted is greatπ!
Happy
<coding />
π!Marked as helpful1@lazzsamPosted over 1 year ago@frank-itachi Thank you very much, i'll check how to use those pseudo classes. π
0 - You should wrap the testimonials section inside the
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