@Y-aburob
Sudip Khatri
@SudipKhatri036All comments
- @SudipKhatri036
Great work on your recent project! I really appreciate the effort you put in. I have founded two issues:
- The border countries is static not from api data
- Country Filter in america it is showing there are no countries.
- Here i think you are filtering data according to "America" which should be "Americas" that's why it is showing no countires
Happy Coding !😊
- @Zahid9985What are you most proud of, and what would you do differently next time?
I recently created a website, and my proudest moment was completing it successfully, especially as it involved tackling a few technical challenges on my own. Next time, I would pay more careful attention to div arrangement to improve the layout and overall structure.
What challenges did you encounter, and how did you overcome them?I encountered challenges with div arrangement in my project. It was tricky to structure the layout precisely the way I wanted, which affected the overall design. However, I tackled this challenge on my own, experimenting with different CSS properties and adjusting the layout until it matched my vision. This experience taught me valuable lessons in layout handling and strengthened my problem-solving skills.
What specific areas of your project would you like help with?I would like to help others with arranging elements in their web projects. Having faced challenges with layout myself, I understand how effective arrangement can enhance both design and usability, and I’m excited to share what I’ve learned.
@SudipKhatri036Great work on your recent project! I really appreciate the effort you put in. Why are you adding width to every element and making it small only for mobile ? you should learn about media querys so that you can make styling according to screen sizes and learn about responisve units and its uses remove width from all container elements and use percentage instead. website is looking good but it should be for all devices not only mobile. Learn more about css and Happy Coding!
Marked as helpful - @roidzuhWhat are you most proud of, and what would you do differently next time?
React, Tailwind CSS, mobile-first approach
What challenges did you encounter, and how did you overcome them?none.
What specific areas of your project would you like help with?Any feedback is welcome!
@SudipKhatri036Great work on your recent project! I really appreciate the effort you put in. One suggestion I have is to include a confirmation message after checkout btn clicked. I think it would enhance the project more.
- @eva003nWhat are you most proud of, and what would you do differently next time?
implemented the model view controller design pattern i learnt recently
@SudipKhatri036Well 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 -
- @Watership6@SudipKhatri036
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 - @Mike-Dave@SudipKhatri036
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 - @NatiGech@SudipKhatri036
Just remove the padding of ul and it will be fixed 😊
ul { padding : 0; }
