@asbhogal
Posted
Hi Jose,
Great work! The design matches the mockup nicely and the app is responsive and functions well (I like your coding practices here too with the custom hook and useEffect implementation. Nice.) Just a few things which are worth considering:
- Always locally host your Google Fonts for privacy and performance reasons. Since you're in an npm environment, you can install the typeface as a Fontsource dependency, import the weights require then declare the relevant properties in your styling. Here's the documentation for the Manrope font Link
- Avoid using Create React App, as Meta no longer supports this. For client-side applications like these, switch to Vite.
Hope this helps!
Marked as helpful
@JooseMM
Posted
Hi @asbhogal
Thank you so much for your recommendations, I will start implementing it in my future projects!.