@MelvinAguilar
Posted
Hello there 👋. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
-
To make the alt attribute as useful and effective as possible, avoid using words such as "image", "photo", or "picture" as they are redundant because the image tag already conveys that information. Also the alt attribute should not contain underscores or hyphens. Instead, try to make the description as human-readable and understandable as possible.
- For a photo of a person, use their name as the alt text
If you want to learn more about the
alt
attribute, you can read this article. 📘.
-
You can use the following styles to center the element effectively using either of these two methods: For Grid:
body { min-height: 100vh; display: grid; place-items: center; /* Additional styles if needed */ }
For Flexbox:
body { min-height: 100vh; display: flex; flex-direction: column; justify-content: center; align-items: center; /* Additional styles if needed */ }
Later, you can remove this margin:
.coursecard__container { /*margin: 0 auto;*/ /*margin-top: var(--mrg-tp);*/ }
I hope you find it useful! 😄 Above all, the solution you submitted is great!
Happy coding!
Marked as helpful
@hashiharis
Posted
@MelvinAguilar
Thank you for your valuable feedback and the tip as well :) . I will update my solution as you suggested.