@Douglas-PassigattiSubmitted 23 days ago
JosephMcMorrisII
@JosephMcMorrisIIAll comments
- @JosephMcMorrisIIPosted 23 days ago
Does the solution include semantic HTML? I feel like the overall semantics of the code look fine only thing I can notice is h2 instead of h1 tag. For example in 21 in the index.html I would change that to h1 instead of h2.
Is it accessible, and what improvements could be made? The code is accessible. Overall all the code seems fine. Just some css sizings could be changed such as the padding around the container and some text/image sizings.
Does the layout look good on a range of screen sizes? I wasn't able to check the other screen sizes
Is the code well-structured, readable, and reusable? I think the code is fine
Does the solution differ considerably from the design? It looks like some css sizings are a little off but it looks very close.
Marked as helpful0