Design comparison
SolutionDesign
Solution retrospective
Did this in less than 30min.
Used grid and flexboxes.
Community feedback
- @grace-snowPosted over 3 years ago
Looks good 👏 But I don't recommend rushing 😉
- Content is hitting the sides of the screen for me on mobile and on mid-sized screens
- Use css to display things block, not <br>s. They get announced by screenreaders, and it's unlikely you would have that level of control over content
- consider using string tag for bold yet instead of span wrapping normal weight text on heading
- consider changing the layout for a mid-sized screen. There is a lot of space and very narrow content on some screen sizes. 320px is very narrow to restrict everything
Great job overall though, keep going
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