Bravin Murega• 10
@Murega14
Submitted
Enoch Beamahn
@djamkenny
All comments
- Enoch Beamahn• 80
@djamkenny
Posted
Your code: body { font-family: "Outfit", sans-serif; font-weight: 700; font-size: 16px; line-height: 1.6; color: hsl(220, 15%, 55%); margin: 8; background-color: hsl(212, 45%, 89%); }
But you can add this: body { display: flex; justify-content: center; align-item: center; }
also, I think you should not use line-height in the body because you are only targeting the <p> and or <h1> elements.
I have noticed you used <div> all and I suggest reading about new semantic elements which is HTML5...it awesome check it out
Marked as helpful
0 - Momshad Islam• 10
@Shad-Islam
Submitted
Kindly share your feedback with me. It would be highly appreciated.
Enoch Beamahn• 80@djamkenny
Posted
I think you have a high border-radius i think 20px could be more better. But you have done so well i checked your responsiveness and it cool
Marked as helpful
0 - ErfanTardast• 40
@ErfanTardast
Submitted
still have problem on mobile size if you have any solution , im glad to read that
Enoch Beamahn• 80@djamkenny
Posted
The are several ways to tackle mobile size, either you choose the mobile first way or you use @medial screen {} where you set the screen size and write the code to repond to the size.
0
