Design comparison
SolutionDesign
Solution retrospective
Feedback Welcome!
Community feedback
- @Zy8712Posted 12 months ago
Your site looks pretty good. Main things you should change are:
- adding a
<main>
tag around your page's main content for accessibility purposes - adding
hover
effect for your "change" button - centering your container using flexbox instead of what you are using right now. You can do this by assign something like this to your body:
body{ display: flex; flex-direction: column; justify-content: center; align-items: center; height: 100vh; }
For more one flexbox you could watch these tutorials: link1 link2.
Hope you find this feedback useful 👍
1 - adding a
Please log in to post a comment
Log in with GitHubJoin 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