@Bayoumi-dev
Posted
Hey Ashutosh, It looks good!... Here are some suggestions:
Document should have one main landmark
, Contain the component with<main>
.
<main>
<div class="container flex-center">
//...
</div>
</main>
-
Add
cursor: pointer;
to thebutton
, The cursor indicates to users there is an action that will be executed when clicking on it. -
Use
REM
for font size, It is a must for accessibility because px in some browsers doesn't resize when the browser settings are changed... See this article ---> CSS REM – What is REM in CSS?
I hope this is helpful to you... Keep coding👍
Marked as helpful
@ashutoshbisoyi
Posted
@Bayoumi-dev Thank you for the helpful suggestions. I'll bring them to practice