P
Matt Studdert• 13,611
@mattstuddert
Posted
Awesome work on this, it looks great! Here are a few pointers after looking at your code:
- Be sure to only ever use one
h1
on a page otherwise, the content hierarchy can be messed up for people using screen readers - For the call-to-actions, you don’t need to have a
button
wrapped around the anchor tag. Just keep the anchor tag and style it directly. - The images for the curved sections could use
::before
or::after
pseudo-elements with abackground-image
instead of animg
tag. These are purely decorative images and so shouldn’t necessarily be in the HTML. Here’s a link to read up on these some more
Hope these pointers help. Keep it up! 🙂
0