Submitted about 1 year ago
Single price grid component using CSS Flex
@franklynxchill
Design comparison
SolutionDesign
Solution retrospective
Hello, Frontend Mentor community π, Iβm Franklynxchill and this is my solution for the Single price grid component . π Any suggestions on how I can improve and reduce unnecessary code are welcome! Thank you. πβοΈ
Community feedback
- @MelvinAguilarPosted about 1 year ago
Hello there π. Good job on completing the challenge !
Here are some suggestions to further improve your solution:
- Use
<ul>
and<li>
for the content under "Why Us": Since the content is a list of items, it's more semantically correct to use an unordered list (<ul>
) and list items (<li>
) instead of just using paragraph (<p>
).
- Consider using
min-height: 100vh
rather than combining heightheight: 130vh
andheight: 100vh
. This approach is more straightforward and ensures that there are no unexpected issues on smaller screens. It simplifies your code while achieving the same result.
I hope you find it useful! π
Happy coding!
1@franklynxchillPosted about 1 year ago@MelvinAguilar Thank you for your feedback. I really appreciate
1 - Use
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