IryDev• 1,580
@IryDev
Posted
Hi @Preciousglows 😄. Congratulation on successfully completing this challenge ! !
I have some recommendations in order to improve your solution :
- Wrap the whole main content of your page into the semantic <main> tag. -Replace the <div class="attribution> by the semantic <footer> tag
- Semantic html improve accessibility of your web page
- Your page should have a <h1> tag
- Replace the <p> tag by the <h1>
- Add to the <main> the property, display: flex;, justify-content: center;, align-items: center; to make the card center on the vertical axis and horizontal axis
- The padding of your card seem too big reduce it to 1em
- To make the corner of the card rounded use the property border radius
HTML :
<body>
<main>
<div class="container">
<div class="pardiv">
<h1>Improve your skills by building projects</h1>
</div>
</div>
</main>
<footer class="attribution">
</footer>
</body>
CSS :
main {
height: 100vh;
display: flex;
align-items: center;
justify-content: center;
}
I hope you'll find this helpful! 😄
Marked as helpful
0
Precious Worukwo• 40
@Preciousglows
Posted
@IryDev Thank you so much!
0