@the-siavash
Submitted
@arshGoyalDev
@the-siavash
Submitted
@arshGoyalDev
Posted
Hey there, nice solution
few suggestions
@scriptkid03
Submitted
@arshGoyalDev
Posted
Nice solution,
@SaidBC
Submitted
@arshGoyalDev
Posted
Hey there,
few things you could improve on
justify-self: end
for products, use the extra space to make the cart and products largerMarked as helpful
What are you most proud of, and what would you do differently next time?
That i was able to add my items to my cart and it's working the way I was told to do
What challenges did you encounter, and how did you overcome them?
when the pop out component that is showing on the desktop is not suppose to show on the mobile layout and some of the arrangement of mobile is different to the Desktop layout, it was really hard for me to get, but I keep trying and I am happy I was able to overcome it
What specific areas of your project would you like help with?
i don't know yet, but if my bosses check it out and see what I need to do, please I will be glad if the feedback can come in
@arshGoyalDev
Posted
Hey, it's a great solution
few things you could improve on
Marked as helpful
@3lectrologos
Submitted
What are you most proud of, and what would you do differently next time?
Some full-stack highlights
✔️ Auth.js (v5) | Google & Github as auth providers
✔️ Prisma + Neon | Store TODOs per user
✔️ Framer motion | Drag'n'drop list items and animations
@arshGoyalDev
Posted
Very amazing solution,
sign in is a great addition,
you could also add overflow: scroll;
to todo's container to make it look even better
Marked as helpful
@dmitinev
Submitted
@arshGoyalDev
Posted
hello 👋 Dmitry,
couple of tips,
max-width: 1470px
use padding to make it better@rushabhhere
Submitted
I am not sure if the manner in which I have added 'height' to the bars in the chart is correct. It worked for the given data but how can I ensure that the chart scales as the data scales?
@arshGoyalDev
Posted
You have done it great
id="monday amount", id="tuesday amount"
and more@HenriqueDummer
Submitted
Hi everyone
Here is another chalenge completed!
If you have any tip or advice, please leave a coment
@arshGoyalDev
Posted
There are a quite a few issues in your solution,
box-shadow: 5px 5px 40px rgba(200, 200, 200, 0.4)
Marked as helpful
@de-sipher
Submitted
Two things; 1) Managing multiple CSS/ JS files for different HTML files is quite difficult and I need to find a more scalable way to approach this problem. 2) Speed. The fetching and loading takes a considerable amount of time which I need to find a way to reduce. The images, the filtering and the querying are the major culprits. My best guess is to take all the data from the API and store it as a local object or in a custom asset folder which will be updated every now and then.
@arshGoyalDev
Posted
It's great but a few fixes would make it better:
@axlgomez
Submitted
I will appreciate any suggestion to improve this project.
@arshGoyalDev
Posted
Great work, It could be more great if you just center align the calculator.
Marked as helpful
@aymen-ben-massoud
Submitted
@arshGoyalDev
Posted
There are some issues in the app:
@JimmyHoang296
Submitted
My friend said to me: "Done better than perfect" So here it is, my first intermediate solution. I have a lot of problem with it:
any advice would be valuable to me. Thank you.
@arshGoyalDev
Posted
There are a lot of issues in the app:
console.log()
in the js file.All the other features are working well, and it would be nice if you add some animations
Marked as helpful