Bryan Li• 3,550
@Zy8712
Posted
Your site looks pretty good. The only things I'd add/modify is a <main>
tag to wrap around you page contents (for accesibility purposes) and using <h1>
instead of <h2>
. This is because headers are suppose to be used going from <h1>
to <h6>
without skipping any. All these headers come with default font sizes that can be changed using the font-size
property in css.
Hope you find this feedback useful 👍
Marked as helpful
0