Design comparison
SolutionDesign
Solution retrospective
Hi everyone i just finished this challenge, i would really love to hear your thoughts on the challange, thank you.
Community feedback
- @grace-snowPosted almost 4 years ago
Hi, This is nicely responsive and looks pretty close to the design. I think the headings are slightly too small and thin viewing on mobile but that's all.
There are some minor improvements I'd make to the html:
- the lower headings are all subheadings under 'join our community' so if the parent heading is a h3, they need to be h4s.
- the price shouldn't be a heading, but again if it is, it has to be another level down, belonging to pricing. In this case that would be a h5.
- in why us, if you find yourself putting lots of line breaks in a paragraph, and it doesn't read like a paragraph, it should be a list. (So swap for one ul and lis for each item)
I hope that's helpful, upvote if it is ☺
2@Igwe0001Posted almost 4 years ago@grace-snow Hi thanks for your input, made some changes to it, thanks for helping again.
0 - @Lazizbek1997Posted almost 4 years ago
Good job man👍
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