ِAbdullah• 730
@9dorgham
Posted
Good work everything looks good.
- You used the "<main>" element to contain your hole content and that's good, but it's a simple card you'd better use container elements like "<div>".
- Why you used this structure "<button class="btn github">GitHub</button>" it's an unorder list of links. We should use the correct elements for the correct purpose you could do this "<ul><li><a href="#">GitHub</a></li></ul>" .
- Use the "<strong>" element for important text like the address "London, United Kingdom" it will not just make it in bold it's important for search engines and assistive technologies like screen readers, you should read about it if you want to learn more. You've done awesome work, have a nice coding day :)
Marked as helpful
0
