Bryan Li• 3,550
@Zy8712
Posted
Hi there! Your site looks pretty good and mobile responsiveness works as expected.
Some suggestions I have are:
- using a
<main>
tag to wrap around the main content of your page - remember to import your font using Google Fonts. Right now your site is just using a default font. Google Font Link: Here
- if you wanted to center your container I suggest you to use flexbox instead of position absolute. In your case you could do:
body{
display: flex;
flex-direction: column
justify-content: center;
align-items: center;
}
And then you could remove your position: absolute
stuff from your container's css. Hope you find this feedback useful 👍
Marked as helpful
0