Here's some suggestions
- Your name should be a h1 I think. Every page needs a h1 and that would be appropriate for it. Alternatively add a visually hidden h1 above.
- The text that starts "Aspiring..." needs to be in a paragraph.
<q>is not a meaningful element on its own. To be honest it doesn't read like a quote at all. - If you are going to make links open in a new window you must notify users, especially screen reader users, of this behaviour. Usually this is done with some visually hidden text inside the link that says "(opens in a new tab)". Optionally you could include an icon to indicate the non-standard behaviour to sighted users as well.
- The cursor style should be on links by default not only when hovered.
- The img border radius can be set as 100% for circles.
- There is no need to wrap the heading and paragraph in a div. Only add necessary html.
0
matbac85• 600
@matbac85
Posted
@grace-snow
Again, Grace, thank you for your thorough feedback. I will correct my code tomorrow following your remarks. Thank you very much. I am learning a lot thanks to you.
0
