Bryan Li• 3,550
@Zy8712
Posted
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