Evan• 180
@eva003n
Submitted
What are you most proud of, and what would you do differently next time?
implemented the model view controller design pattern i learnt recently
Sudip Khatri
@SudipKhatri036
All comments
- Sudip Khatri• 450
@SudipKhatri036
Posted
Well Done! Your solution looks great. These two minor fixes will make your project more perfect
-
.add-to-cart, .update_quantity { transition: all .5s ease-in-out; // this is making ui experience feel like slow and laggy reduce this value to make user experience better } *while adding item to cart button state is taking more time that's making it feel like slow .
-
While removing one cart item after adding two or more items the state of added items is not staying same in increase decrease state but it is going back to add to cart state. You can go through my solution for idea it is not that great but i think you will get idea for this.
Happy coding! 🙂
Marked as helpful
0 -
- Watership 🌊• 320
@Watership6
Submitted
Hey, I am Jet Black also known as Crow🙋♂️. And this is my solution for the expenses chart challenge. 🚀
Goals Accomplished ✅:
- I was able to get better with svelte and tailwind css
- I got more practice with building dark and light modes properly
Built With🪄:SVELTEJS✔️TAILWINDCSS✔️
Overall, I really liked this challenge, and would appreciate any feedback that's given. ❤️JetBlack12 🙋♂️
Sudip Khatri• 450@SudipKhatri036
Posted
I looked at your website its all good but one thing is missing and that is its responsiveness . You should focus on making the website responsive . In mobile view it doesn't show the theme changer button and the website is not fully showing in mobile screen . Happy Coding!
Marked as helpful
1 - Michael Chukwu• 360
@Mike-Dave
Submitted
I successfully completed this challenge, which proved to be an incredibly rewarding experience. I welcome any feedback and recommendations you may have. Thank you for taking the time to provide them.
Sudip Khatri• 450@SudipKhatri036
Posted
You didn't changed the email in success message all are good . The success message shows static email . And your image is not taking full width in mobile design . Happy Coding !
Marked as helpful
1 - Skat-16• 140
@NatiGech
Submitted
Coludn't fix the distance between the lis images and the labels.
Sudip Khatri• 450@SudipKhatri036
Posted
Just remove the padding of ul and it will be fixed 😊
ul { padding : 0; }
1
