@JamesLaviron
Posted
- Does the solution include semantic HTML?
Yes it does, nice ideas. Loved the way you used classes and variables.
- Is it accessible, and what improvements could be made?
yes it is accessible. I would pack texts in a container that handles most of padding stuff.
- Does the layout look good on a range of screen sizes?
It looks good.
- Is the code well-structured, readable, and reusable?
It seems so, I would personally have sticked with HSL.
- Does the solution differ considerably from the design?
Yes, a lot. Are you using figma? It seems like many values are approximative. Overall it's looking nice :)
Marked as helpful
@yasminsilva1
Posted
Hi, @JamesLaviron! Thanks for your feedback! No, I didn't use figma. I made the page looking at the preview and tried to reproduce it as faithfully as possible.