@domieee
Posted
Hi Abdoulaye
I took a look at your code and found a few things you could adjust.
you have given the img
element a height and width of 180px. This would be unresponsive if necessary and would not adjust to its parent element. You could fix this by giving the container a max-width
of the desired width and the width: 100%;
. The img
should also have the property width 100%
.
for web accessibility you should add a <main role="main>"
and nest your code in it. Also you should change your h3
element to an h1
element. this is also important for this case. You can take a look at google and see what is important for web accessibility.
try to use rem
instead of px
for the font size. if necessary the font's wouldn't adjust to the brower size with px.
Keep coding and have a great day
Marked as helpful
@domieee great feedback but no need to put role of main on a main element. It already has that role
@domieee
Posted
@grace-snow well, my bad. thanks for the correction.