@amalkarim
Posted
Hi Enn Ko, congratulations for completing the challenge!
Please check at this line:
<img src="./images/image-hero-mobile.png" class="w-100" style="height: 450px;" alt="">
I think it's better that you only set its width using class="w-100"
, or only set its height using height: 450px;
. Don't use them both, because it will make the image not proportional in mobile view.
We could also add alt
for the hero image because it's an important image and adding alt
will make it more accessible to all users. For example:
<img src="./images/image-hero-mobile.png" class="w-100" alt="A curly hair man standing with a laptop on his hand in front of a yellow wall">
Please refer to this article from W3Schools about image accessibility for more explanation.
Hope this helps. Happy coding!
Marked as helpful
@enn-ko
Posted
@amalkarim Thank you for your suggestion and I fix it.