Design comparison
SolutionDesign
Solution retrospective
Hope you like it!
Community feedback
- @RioCantrePosted almost 3 years ago
Hello there! Great work with this one. Regarding the solution you submitted, I think you should also take notes of the following…
- Instead of
div
, alternatively wrap the whole content in the App component withmain
tag. Use semantic tags, for HTML structures, refer it with this one Semantics - Include
alt
with description in image tags. Like this line<img className="divider" src='./images/pattern-divider-desktop.svg'/>
Advice I got:
Advice # 63
"If you're squashed close to strangers on public transport, try not to be rude to them. No one likes those situations."
Above all, The design looks good. Keep it up!
0 - Instead of
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