Hi Dimitri,
congratulations in submitting this solution, good job!
I had a look into your solution and noticed a couple of things that could be optimised.
- your HTML follows a clear and standard structure, which is really good!
- You also used semantic headings which is great for accessibility and SEO.
- you have linked twice the preconnect. You could remove one and only keep one.
- your image is missing an alt text which is important for accessibility and SEO.
- there is an excessive use of <br> tags for spacing, which is generally discouraged. CSS is usually a better solution to handle spacing.
- the table does not follow exactly the design that was provided to us. I can see you also created div, but here I would recommend to use the table tag. You can add the missing border by adding a border bottom on the td.
- your page currently looks nice on desktop, but is not responsive and adjusting well on smaller screens. I would recommend implementing media queries for this.
I hope this is helpful!
Keep up the good work :)
0
