Hello
You’ve done pretty well here, but the contents is too wide for my mobile. Here are some tips
- if an image is decorative, it doesn’t need an alt value. You can leave it blank like alt=“”
- you don’t need to wrap links in paragraphs. If you are going to do that, make it consistent and do it every time.
- optional, but I expect “change” would perform an action, so should be a button element not anchor tag
- your media query is doing nothing, so you can remove it.
- Biggest tip for you is to get out of the habit of giving elements explicit widths and heights, especially height. Let things be the size they need to be, as dictated by the content and it’s paddings/margins. Use max width and min height only where needed. This will build more responsive solutions - ie if clients change the amount of text, if users view on an unexpected screen size or change their font size etc
I hope this helps
Marked as helpful
0
Romel Williams• 180
@omerome83
Posted
@grace-snow Thank you very much! I really appreciate all of these tips!
0
