@Alex-Archer-I
Posted
Hi!
A couple suggestions about semantic tags.
- It's a list, so it's better to use
ul
tag here. - It should be only one
h1
tag on the page. Guess since this project is out of context it's fine to skiph1
tag at all. - And I'm not get quite familiar with this topic, so I can't tell for sure, but seems that when text is a quote it should be inside
blockquote
tag. Here the web.dev article about it.
I guess you could make a mobile version a bit easier =) You can change the grid template to grid-template-column: 1fr
instead of changing every item inside it.
Oh, and you could also make the quote image with before
or after
. It'll spare from fiddling with background properties, but you have to fiddle with positioning and z-index, so I don't really know what is better =)
It's very cool work, keep doing =)
Marked as helpful