@amalkarim
Posted
Hi Nicholas, great job on completing this challenge.
I never used Tailwind though, so I couldn't really give you suggestion about that. I think your code is great. The only thing that I found could be improved is the height of the <main> element. When the page is viewed using a mobile phone in a landscape mode (i.e. the height of the browser is shorter than the content), some parts of the card in invisible, and we can't scroll to see it. To avoid that, you can simply remove the class h-screen from the <main>. Also, try to give the body some padding to give a nice breathing room for the card.
Hope this helps
Marked as helpful
@NicholasAnich
Posted
@amalkarim Thanks for the feedback! I put h-screen (full vh) on the main so I could center the component in the middle of the screen haha. Would it have been better to use position in this case? Or should maybe flex-box on the body? I'll work on it! Cheers man!
