Design comparison
SolutionDesign
Community feedback
- @BlackpachamamePosted 11 months ago
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 helpful1@IgorGGuimaraesPosted 11 months ago@Blackpachamame Thanks for the help! Really appreciate it!
1 - Use semantic tags such as
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