Ahmed Bayoumi• 6,740
@Bayoumi-dev
Posted
Hey Damilola, It looks good!...
You have some accessibility issues
that need to fix.
Page should contain a level-one heading
, Changeh4
toh1
You should always have oneh1
per page of the document.- This
<div class="dice"></div>
Should bebutton
...Buttons
are used for actions, and I suggest you addcursor: pointer;
to thebutton
, Thecursor
indicates to users there is an action that will be executed when clicking on it.
I hope this is useful to you... Keep coding👍
Marked as helpful
0
Damilola Mohammed• 230
@activus-d
Posted
@Bayoumi-dev thanks for the feedback
0
Damilola Mohammed• 230
@activus-d
Posted
@Bayoumi-dev I've made the required corrections now
0