Mark Cates• 110
@mscates
Posted
Hey JC, the one accessibility issue you have is the h2, they want at least one h1 on each project we do. not a big deal, but if you want that to go away that is the solution.
another thing I see is you could also make a card class that wraps the entire content and put a max-width on it of say 400px and then width of 100%. that way no matter how big the screen gets, it will only be 400px. right now, I have a large screen and it stretches your card out quite a bit because it doesn't have that limit. everything else looks great though. nice work.
Marked as helpful
0
JC• 150
@jcsmileyjr
Posted
@mscates Thank you for the wonderful advice.
0
