Design comparison
SolutionDesign
Solution retrospective
Feedbacks are welcome
Community feedback
- @AdrianoEscarabotePosted about 2 years ago
Hi Samarth Goyal, how are you?
I really liked the result of your project, but I have some tips that I think you will like:
1- Every pages needs a
<h1>
to show which is the most important heading. So replace the<h2>
with<h1>
and follow the sequence h1-h5I noticed that at higher resolutions, the background image is broken, to fix this we can do the following:
.order-component { background-size: 100% 50vmin; }
The rest is great!!
Hope it helps...👍
0@geekSamarthPosted about 2 years ago@AdrianoEscarabote Thanks Adriano for your appreciation and valuable suggestion, it will help me to improve my CSS skills and surely I will definitly work over it.
1
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