Responsive single Price grid using html avd css
Design comparison
Community feedback
- @CodeWithAlaminPosted almost 2 years ago
Hi bisratworku👋 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: Instead of using generic tags like
div
andspan
, the developer could use semantic tags likeheader
,article
, andfooter
to make the code more meaningful and easier to understand. This will also improve the code's accessibility for users with disabilities. -
Refactor repetitive styles: To make the code more efficient and maintainable, the developer should consolidate styles that are repeated in multiple places into a shared class or a global style rule.
-
Use more descriptive class names: The developer could use more descriptive and meaningful class names like
.section
or.pricing-section
instead of short and generic names like.sction
. This will make the code more self-explanatory and easier to work with.
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