Here’s my tribute page and the first task I’ve ever completed. I’d love your feedbacks
https://codepen.io/ShAfiii/full/xxGBWpN
4 Likes
Your first project looks very nice. I especially like the link styling. However, I would like to remind you that, for the screen readers, we should not put links to “read more” parts. It makes nonsense for screen readers. It’s better to link the “To know more about Marie Curie” part.
1 Like
Thank you for your advice
Hi @ShAfi, your page looks good. Some things to revisit;
- When using codepen it only expects the code you’d put within the
<body> </body>
tags in HTML. (No need to include the body tags). For anything you want to add to<head>
click on the ‘Settings’ button, then HTML and add it into the ‘Stuff for <head>’ box. - You have a duplicate
id
. Anid
should be unique within the document. - Review the lesson about giving meaningful text to link.
- Web Accessibility in Mind gives a more thorough explanation
- ‘click here’ is not accessible
1 Like