@visualdenniss
Posted
4 - Add cursor: pointer; to the interactive elements like the links or the overlay/img to indicate interactivity to the user.
Marked as helpful
I would love to see some suggestion to keep improving
@visualdenniss
Posted
4 - Add cursor: pointer; to the interactive elements like the links or the overlay/img to indicate interactivity to the user.
Marked as helpful
@DXXDLY
Posted
Great work!
Here are some changes to your code
2.Use more declarative class names. The class names like .card, .img-container, .creation do not give any idea of what these elements represent. It is recommended that you use more specific class names that more accurately describe the elements.
3.For better readability, it makes sense to group CSS properties by type, for example defining font properties first, then colors, and so on.
Hope you find this feedback helpful!
Marked as helpful
@mohamed-magdy26
Posted
@DXXDLY I appreciate your help. It's so beneficial, but isn't the alt attribute for the image a decorative image? I should leave it empty.
@DXXDLY
Posted
@mohamed-magdy26
The "alt" attribute is used to describe the content of an image on a web page and is intended to improve the site's accessibility for people with disabilities. And It can also be useful for search engine optimization and user experience.
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