Tailwind Css, HTML, JS

Solution retrospective
It took me a lot of time to make this project as it was my first time using tailwindcss(really!!).
So, any suggestions/tips to improve are highly appreciated!😊
Please log in to post a comment
Log in with GitHubCommunity feedback
- @pklepa
Really well done, mate! I've never user Tailwind before but I thought your mobile menu animation was quite clean.
There are a couple of things that I noticed that if addressed, could make your implementation even better:
-
Your fonts are overall a little small. The Style Guide of the challenge states that the base font-size should be 18px and it really makes a difference to properly take advantage of the provided typography.
-
On a similar note, your max-width for the content is set to 1536px instead of the suggested 1440px from the styleguide. (On my implementation I used 1240px as I found it to be little neater)
-
The way you handled the API errors is super cool and I think I'm going to copy to my own version, but you should probably fix the spacing for long messages as it is overlapping with the button on smaller screens.
Great job overall! Cheers!
-
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