@elaineleung
Posted
Hi Julia, congratulations on completing your first project! I just had a look and it looks pretty good, so well done on getting it to close to the design. You do have a number of accessibility issues and HTML issues in your report, so be sure to check those out and try to fix that. Some of them concern the lack of semantic tags (or landmarks), like the <main>
tag, which you can add like this:
<body>
<main>
content here!
</main>
</body>
Also, I notice you're using fixed widths; you can try using responsive width properties instead, like max-width
; that way, your component can be resized by the browser on smaller screens.
Great job, and keep coding!
Marked as helpful
@youlia121
Posted
@elaineleung Thank you for your helpful answer, I will definitely take it into account next time!