I used rotate for active state

Solution retrospective
Help
Please log in to post a comment
Log in with GitHubCommunity feedback
- @Hassiai
replace <div class="main"> with the main tag , <div class="attribution"> with the footer tag and give the alt a value <img src="icon-view.svg" alt=""> , to fix the accessibility issues.
To center a content on a page, add min-height:100vh; display: flex; align-items-center: justify-content: center; to the body. There is no need to give .container a height value.
Use rem or em as unit for the padding , margin, width and preferably rem for the font-size for more on this watch this https://youtu.be/N5wpD9Ov_To
In the media query there is no need to give the body, a width value, a height value and color value. Give .container a width of 80%, this will help, there is no need to give .container a margin value.
Hope am helpful HAPPY CODING
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord