@PhoenixDev22
Posted
Hello @sirargill,
I have some suggestions regarding your solution:
-
The images don't need filled
alt
. They are decorative images, so alt attribute should be left intentionally blank. You can optionally addaria-hidden
orrole presentation
to ensure the images are always ignored by screen readers ANDavatar's one andimage-equilibrium.jpg
should not be empty and should be descriptive . Read more about decorative and informative images . -
You can simply use unordered list
<ul>
to wrapclass="side-align’ and in each list item would have
<img >and
<p> `. -
You don’t need
<hr >
, you can useborder-top
for theclass="side-align
. -
You can use more html semantic tags like
<figure> <figcaption >
for the avatar part. -
Add the hover effect on the image . (The eye image is missing)
Hopefully this feedback helps.
Marked as helpful
@sirargill
Posted
@PhoenixDev22 thank you for your kind suggestions, I will make sure to implement those.