@beowulf1958
Posted
Congratulations on completing this challenge! It looks great and is responsive.
Two suggestions: I notice you use heading tags h2, h3 for styling. This is not a good practice; h2 is for a sub-heading of h1, h3 a subheading of h2, and so on. This can confuse screen readers.
Also, the design called for an active state for the button: on active, the cursor: pointer and the background becomes a linear-gradient similar to the circle.
Your code is very clean and readable. Hope this helps.
Marked as helpful
@CaseyLawver
Posted
@beowulf1958 Thank you for your feedback! Would you suggest using p for those headings instead? I completely forgot about the active state for the button! I'm going to add that, thanks for pointing that out.
@beowulf1958
Posted
@CaseyLawver Yes, <p>. I reversed engineered your project with p tags and it worked fine. You have to specify first-of-type, last-of type, but otherwise your styles work.