Design comparison
Solution retrospective
Is it possible to receive feedback from your community?
Community feedback
- @denieldenPosted about 2 years ago
Hello Nada, You have done a good work! π
Some little tips to improve your code:
- add
main
tag and wrap the card for improve the Accessibility - use
blockquote
tag to wrap the card and improve the Accessibility but not as a container of that one element - also you can use
article
tag instead of a simplep
for the advise - instead of using
px
use relative units of measurement likerem
-> read here
Keep learning how to code with your amazing solutions to challenges.
Hope this help π and Happy coding!
Marked as helpful2 - add
- @md5daltonPosted about 2 years ago
Greetings Nada Elhosary π
Congratulations on completing this challenge. I also enjoyed working on it, especially fetching and using the data from the API.
You may have noticed that in firefox browser there's an issue where no new advice is fetched even if you keep pressing the dice. This is due to the fact that the browser is caching the previous response, so to prevent this, you can add a "no cache" option to your fetch function:
fetch(url, { cache: "no-cache" })
. This will ensure a great user experience for everyone using your app regardless of which browser they are using.Another minor thing I noticed in your HTML markup is you implemented responsive images with the
picture
element and that is greatπ, however, you included<source media="(max-width: 520px)" srcset="images/pattern-divider-mobile.svg">
and this is redundant since<img src="images/pattern-divider-mobile.svg"/>
loads the same image. You can remove it and your code will work perfectly fine.Hopefully that helps improve your project π
Marked as helpful1
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord