Danny Gomes• 120
@dannygomes
Posted
Hey! It looks pretty clean!
The one suggestion I can give is to put all that content inside a <main> tag. Someone gave me that tip on my first submission so the first thing I do now whenever I start a challenge is add that <main> tag in and add/work on all my content in there that isn't a navbar or a footer.
You got the line height right by the looks of it! I didn't notice the line height on my first submission.
Keep it up!
Edit: Just a quick disclaimer that I am a beginner so if I said anything wrong, please do correct me!
Marked as helpful
0
Mo'men Ashraf• 340
@Moamen31
Posted
@dannygomes Thank you Danny, I appreciate your feedback and I will for sure keep that in mind in my next projects.
0
