Marcos Travaglini• 4,920
@Blackpachamame
Posted
Good job!
Here are some comments that may help you:
- Use semantic tags such as
main
,footer
, etc, this helps the accessibility and SEO of the site - The
<div class="main">
should be<main class="main">
- The
<div class="attribution">
should be<footer class="attribution">
. Place the footer outside the main - With these changes, the current main is no longer necessary
- To center the elements in the center of the screen you can make the following adjustments in the
body
:
body {
background: linear-gradient(to bottom, var(--light-grayish-blue), var(--light-gray));
background-repeat: no-repeat;
background-size: cover;
width: 100%;
min-height: 100vh;
display: grid;
place-content: center;
}
Marked as helpful
1
IgorGGuimaraes• 120
@IgorGGuimaraes
Posted
@Blackpachamame Thanks for the help! Really appreciate it!
1