@echocode1
Submitted
What specific areas of your project would you like help with?
all review and inputs is welcome
@ipaulaa
@echocode1
Submitted
What specific areas of your project would you like help with?
all review and inputs is welcome
@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
@popimi
Submitted
@ipaulaa
Posted
You did an excellent job with the dimensions and positioning. However, the colors and font do not match the specifications. The correct colors and font are provided in the README.md, so you don't need to guess them. Additionally, consider moving the styles defined in the
Marked as helpful