Design comparison
SolutionDesign
Community feedback
- @anoshaahmedPosted almost 3 years ago
To avoid accessibility issues in the future:
- wrap everything in your body in <main>
- start your headings with h1 and move up by one level each time
Good job! :)
1 - @A-amonPosted almost 3 years ago
Hello! Awesome work on this~ π
Here are some tiny suggestions:
- You don't need
<br>tags for the paragraph. Let the words wrap naturally. - To ensure the card is in the center horizontally on large devices, unset width for
<body>. Then, setbackground-size:100%;so that the background image fills up the remaining space.
1 - You don't need
- @meladcodesPosted almost 3 years ago
Hi , Well done on completing it. In order to avoid accessibility issues you can wrap everything with a <main></main> tag. Also you don't have to write your width and height in strings.
Hope this helps!
1
Please log in to post a comment
Log in with GitHub
Join 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