single-proce-grid-component-main
Design comparison
Community feedback
- @CodeWithAlaminPosted almost 2 years ago
Hi Arun Muralidharan👋 Great job on completing this challenge! 🥳
I would like to share a few suggestions on this solution if you don't mind.
-
Use semantic HTML tags whenever possible to improve the meaning and accessibility of your content. For example, instead of using a
div
element for the banner, you could use aheader
element. Similarly, you could use anul
element for the "Why Us" list instead of adiv
. -
Consider organizing your CSS rules into logical sections or blocks, using comments to separate them. This can help make your stylesheet easier to read and maintain.
-
Make use of the CSS cascade and inheritance to avoid repeating styles and reduce the size of your stylesheet. For example, you can define a style for the
body
element and have it apply to all child elements, rather than defining the same style for each child element individually.
Overall, this is a very well done solution to the challenge. Great job!
I hope this feedback was helpful. 😊 Keep up the good work!👍
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