@ipaulaa
Posted
Does the solution include semantic HTML? No, you should take a look at Semantic HTML it makes it easier to style and understand the code.
Is it accessible, and what improvements could be made? It makes use of alt tags for images with is great.
Does the layout look good on a range of screen sizes? Yes, it's responsive
Is the code well-structured, readable, and reusable? It's preferred to use class for styling then ids.
Does the solution differ considerably from the design? The size is a little off. And there is no :hover effect as is asked from the brief of the challenge.
Great job. Just a few thinks that need improving!
Marked as helpful