romila• 3,570
@romila2003
Posted
Hi Bruno,
Congratulations 🎉 for completing this challenge, your API component looks great, and it is great that you used flex
to center the card. I have some suggestions I want to address:
- It is best practice to wrap the main content within the
main
tag which would ensure that your content is wrapped within the correct landmarks e.g.<main class="container"></main>
- Since you have a
max-width
on your card, there is no need to use a media query. - Even though your API is functional on normal browsers, it won't work on Firefox without the clear cache e.g.
fetch(URL, {cache: 'no-cache'})
Overall, great work and wish you the best for your future projects 👍.
0