@Zy8712
Posted
Your website look great. Good job!!
The only thing I feel like you should change is to use <h1>
tags for the title of the component instead of the <p>
that you are using right now. The main reason being that screen readers may look for header tags, which your site is missing right now.
Aside from that your code is well formatted and easy to follow along. Great work 👍
Marked as helpful
@ning-sy210
Posted
@Zy8712 Hey Bryan, thanks for stopping by and leaving a comment! I thought about using the <h1>
tag for the bold text but didn't know if it was appropriate or not. So ultimately I just settled with using a <p>
tag instead. But I will make a note for my future challenges!