@Bayoumi-dev
Posted
Hey Brian, It looks great!... Here are some suggestions:
- Instead round each corner and create media queries to change it based on the screen size, Give the parent these classes
rounded
,overflow-hidden
<main class="container rounded overflow-hidden" id="container">
//...
</main>
- Using more than one
<h1>
is allowed by the HTML specification, but is not considered a best practice. Using only one<h1>
is beneficial for screenreader users.
---> Multiple <h1>
elements on one page
Hope this help!... Keep coding๐
Marked as helpful
@superschooler
Posted
@Bayoumi-dev ahhh, so hidden overflow is the missing piece for me. Thanks!!