Anosha Ahmedβ’ 9,340
@anoshaahmed
Posted
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
Design comparison
Community feedback
Amonβ’ 2,560
@A-amon
Posted
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
Meladβ’ 110
@meladcodes
Posted
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