Hexer• 3,680
@Phalcin
Posted
Wow, nice work man
- Your section lacks heading. Consider using h2-h6 elements to add identifying headings to all sections.
1
Design comparison
Community feedback
Suraj Raju Hadage• 690
@SurajHadage
Posted
Use main element for card. Don't use sections instead use figure element for image followed by picture element. This will shorten your css code. Always use 1 top level heading for every single webpage or simply use role attribute like "role="heading"". Don't upload unnecessary files to save server space. Just upload index.html, style.css and images folder.
0
Please log in to post a comment
Log in with GitHub
Join 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