Udochukwu Amaefule• 570
@UDsGitHub
Posted
Hi Amaka, Your site looks good, there are just a few things you can do to make it work better...
- You should have a
min-height: 100vh;
on your body so it always stretches to fill the entire view height. That would help with aligning the card vertically. - Your card should have a max width... but then again that could make it small on larger screens so dont make the mistake I made of having a max width based on what i could judge from my small screensize.. it seems like the 90% you have now works pretty well, but to make it better, you could use a width of
max-width: max(90%, 25rem)
-- This would basically choose the bigger width between 90% and 25rem, so on larger screens, it would be 90%, but on smaller large screens, it would be 25rem... assuming 25rem is bigger than 90% of the body's width just note the 25rem i put there is arbitrary.. you should put a value there based on what looks like it works for you.
Aside from all my rambling, you are good! 👍
Marked as helpful
0