Agnik Bakshi• 480
@Agnik7
Posted
Hi, Congratulations on completing this challenge. I have some suggestions, which I think will help improve your skill.
- For the footer text, use the
footer
tag instead ofdiv
. - Wrap all the content inside the
body
tag, except thefooter
tag, inmain
tag. - Replace the
h2
tag with anh1
tag. This will get rid of the accessibility issue. - The image can't be viewed, because the code doesn't know where to search for the images folder. To fix that, add a
./
at the start of the source of theimg
tag.
Replace
<img src="images/image-qr-code.png" alt="Qr">
with
<img src="./images/image-qr-code.png" alt="Qr">
- For the container, instead of specifying the
width
, specify only themax-width
. This will help achieve responsiveness, and you will also not need themedia
query.
Hope this feedback helps you.
Have a nice day!!
0