Hi @tbeagle2,
Great job submitting this solution
I was just taking a look at your code and found something you might be interested in, really small details, but thought I'd share them
In the <body></body>
element, its best practice to include a <main></main>
element for screen readers and accessibility, the first ´<div></div> could be swapped for <main></main>
And just a couple of tiny detail in your CSS,
- to add a
cursor: pointer;
to the.question:hover
- adjust the media query around the >400px width to allow more space for the text
All said, it's looking great
Marked as helpful
@Deanogit
Thank you! I originally have a main element but deleted it, it's good to know that it should included. I never knew there was a reason to use main, it seemed interchangeable. I added cursor pointer, I didn't know that property existed. I increased the media query size to 400, the style design had it at 375, but it does seem really small so it's a welcome change. Thanks again for the feedback!