Latest solutions
cursor+react+tailwind
#react#tailwind-cssPSubmitted 3 months agoI used the new cursor agent extensively for this one, what do you think the outcome, I'm curious to know
Latest comments
- P@danmlarsen#react#tailwind-css#typescript#vitest#framer-motionP@niuguy
great solution with tests , thanks for sharing
- @dullarzeeP@niuguy
The layout looks great, and dark mode is working well. Just a couple of thoughts:
- On mobile, the search and filter elements seem to overlap.
- Should the border countries on the detail page be clickable links as well?
- @TrEv0rRrRrP@niuguy
All looks good to me. I learned quite a bit from your solution, thank you
- @ArcloanWhat are you most proud of, and what would you do differently next time?
It was really fun to build a page with state using React. It is the first project where I used state.
What challenges did you encounter, and how did you overcome them?I had a lot of trobule making the dialog on mobile version the same as the design. I endend up making it a little bit different. I used inset-0 on the dialog but it wasn't placed on the entire viewport. I didn't understand why. Instead I had a lot less trouble with vite deploying the site with github pages so it was a real relief.
What specific areas of your project would you like help with?Any help on the dialog, how to make it look similar to the design is really appreciated. It didn't respond like earlier projects where I built the mobile nav with ease using a fixed element and using inset-0. I didn't understand why it didn't work this time. Any other suggestion on my approach with state or anything else is really appreciated. Thank you :)
P@niuguyThe solution overall looks excellent. I noticed a few minor issues to address: on desktop, the '+' and '-' icons on the 'add to cart' button appear to be switching positions. Additionally, it would improve user experience if the confirm order modal could be dismissed by clicking anywhere outside the modal itself.
After all, congratulations on finishing this one! this is not easy, isn't it?
Marked as helpful - @Chayapol-cP@niuguy
all looks good , code is well-structured and readable 👍
A small enhancement could be setting a default tip rate (e.g., 0%) for non-US users:)
- @ArcloanWhat are you most proud of, and what would you do differently next time?
This is the first project I built with react. It was really fun. I find react really enjoyable!
What challenges did you encounter, and how did you overcome them?Nothing in particular.
What specific areas of your project would you like help with?I don't know if it is correct to leave an empty div to let it take the role of root for a react app or is a better way from an accessibility/pratical point of view. Any suggestions are really welcomed! Thank you
P@niuguyIt's good practice to set up components and keep the codebase clean. However, I'm unsure whether it's a good idea to place the components in the assets folder.
Marked as helpful