@elaineleung
Posted
Great work Deborah! This looks really good, and I think the responsiveness is quite well done. I just got two suggestions:
-
At around the 660px breakpoint when the layout changes, the sides of the component are touching the sides of the browser, so I'd probably just add some spacing to keep that from happening (maybe
margin: 1rem
would be good enough!) -
Your report is giving you some issues about missing alt tags, and actually even for decorative images, you still need the alt tag in the
img
, but you can keep it empty, like this:<img scr="image.png" alt="" >
Once again, well done here, and I'm glad you got improve upon your old solution!
Marked as helpful
@debriks
Posted
@elaineleung Hi Elaine! Thank you to have taken the time to give me some feedback! I'm going to correct the code following your comments!