Design comparison
SolutionDesign
Community feedback
- @iddahadevPosted 4 months ago
Does the solution include semantic HTML?
Overral it does, but a few things need to be fixed:
- There are
h2
andh4
but noh1
andh3
. - The main title should be
h1
. - The preparation title should be
h2
. - The
span
to make the text bold should be astrong
orb
.
Is it accessible, and what improvements could be made?
- The
alt
for theimage
should be human readable. - The table row does not have a table header, it should have one with
scope="row"
Does the layout look good on a range of screen sizes?
- It looks a bit small on desktop.
- The layout for the mobile is the same as the one on desktop.
Is the code well-structured, readable, and reusable?
- The HTML is readable.
- The CSS could use a bit of structure with variables and better named classes.
Does the solution differ considerably from the design?
There are differences with the design such as colors, font weights and spacing, but the structure follows the design.
0 - There are
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