Design comparison
Community feedback
- @Akhil-NagpalPosted over 1 year ago
Hello Giulia! 👋 Congratulations on completing this project! 🥳
To fix accessibility & HTML errors, you need to fix these things in your code:
- Always add alternative text to all images you use like:
<img src="illustration-1.svg" alt="girl-listening-music">
. This specifies an alternate text for an image if the image cannot be displayed.
Other Suggestions 📝
-
Use relative units like
em
orrem
instead of using absolute units likepx
. This will improve responsiveness. Read this to understand better 👉 Should we use Rem, Em, or Pixel? -
Background image is not responsive while adjusting the screen size. Use
background-size: cover;
-
You missed some properties in the
body { }
tag. For centering anything use these properties:
display: flex; text-align: center; align-items: center; justify-content: center;
Otherwise! You did a great job! 👏
I hope these suggestions will help you to improve! 😊
Happy Coding!
1 - Always add alternative text to all images you use like:
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