Design comparison
Solution retrospective
It has been a while since i coded. So i'm a little rusty but i will get back in shape soon. Quite new at using node_modules and packages. I hope someone can review this and give some guidelines since it's been long and i had to google 50% of stuff i used to know. Anyway, any comments are welcomed :)
Community feedback
- @grace-snowPosted about 3 years ago
This looks good to me
Only suggestions are
- look up how to write better alt text on images. Eg you don’t need to use words like image when it’s an image tag
- definitely change
alt=“logo”! That’s the most important info on the whole page and you’ve not said the name of the product/site
Good luck on your next one
Marked as helpful1@Vj3koPosted about 3 years ago@grace-snow tnx for advice. i never know how to name a logo when the company name is unclear. What should i put in this one? something like : Monograph dashboard logo ?
0@grace-snowPosted about 3 years ago@Vjekoni yes something like that. Although if it is going to follow the convention of the logo serving as homepage link, it would be wrapped in an anchor tag. Then the alt could serve as the anchor content or the logo hidden and some sr-only text saying something like “Monograph - Home”
Marked as helpful1
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