Skip to content
OverviewSolutions
27
Comments
24
Arsh Goyal
@arshGoyalDev

All comments

  • Arsh Goyal•940
    @arshGoyalDev

    Hey there, nice solution

    few suggestions

    • aligning the order confirmation modal at center would be great
    • a little margin for order confirmation modal for mobile devices would be great touch
  • Arsh Goyal•940
    @arshGoyalDev

    Nice solution,

    • it would look good if the inputs were little bit bigger
  • Arsh Goyal•940
    @arshGoyalDev

    Hey there,

    few things you could improve on

    • on clicking add to cart you should start the item count by 1 instead of 0 it will enhance the user experience
    • for bigger screen instead of using justify-self: end for products, use the extra space to make the cart and products larger
    • in the cart, border for products list should be little bit lighter, that will make it look good
    Marked as helpful
  • Oluwasegun Benjamin Tobi•470
    @Bensaxxy
    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

    ReactJs and TailwindCSS

    #react#tailwind-css
    1
    Arsh Goyal•940
    @arshGoyalDev

    Hey, it's a great solution

    few things you could improve on

    • increasing padding in the header and product info would make it look better
    • making the picture a bit smaller would make it more close to the design
    • and as a addition clicking on checkout btn can clear the cart and showing a message like checkout complete
    Marked as helpful
  • Arsh Goyal•940
    @arshGoyalDev

    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
  • Arsh Goyal•940
    @arshGoyalDev

    hello 👋 Dmitry,

    couple of tips,

    • for mobile menu, you could add a background overlay to separate the menu from the rest of the website
    • for bigger devices instead of using max-width: 1470px use padding to make it better
  • Arsh Goyal•940
    @arshGoyalDev

    You have done it great

    • It seems like your solution could work but you could use "%" instead of pixels.
    • And to fix some of the accessibility issues you should either remove all of the ids you added in amount div or you could add them like id="monday amount", id="tuesday amount" and more
  • Arsh Goyal•940
    @arshGoyalDev

    There are a quite a few issues in your solution,

    • It's not responsive.
    • There is no navbar for small devices.
    • Logo section is overflowing on the learn more btn.
    • When one dropdown menu is open and I try to open another the previous should be automatically closed.
    • The box shadow of the dropdown menu should be smooth, it could be something like this box-shadow: 5px 5px 40px rgba(200, 200, 200, 0.4)
    Marked as helpful
  • Arsh Goyal•940
    @arshGoyalDev

    It's great but a few fixes would make it better:

    • Title in the navbar should be a link to the home page.
    • There are no commas in the population number that make it hard to read you could use regex to fix it
  • Arsh Goyal•940
    @arshGoyalDev

    Great work, It could be more great if you just center align the calculator.

    Marked as helpful
  • Arsh Goyal•940
    @arshGoyalDev

    There are some issues in the app:

    • User cannot add items to the cart.
    • Lightbox functionality is not there.
    • No control buttons for the image carousel on mobile devices.
    • You should make the mobile nav menu opaque so it's visible.
    • You should improve the navbar for the bigger devices (desktops or laptops).
  • Arsh Goyal•940
    @arshGoyalDev

    There are a lot of issues in the app:

    • When I add a comment, the added comment layout is messed up.
    • When I try to reply to a comment, the comment content is in all blue (only username should be in blue),
    • I can't add a reply to a reply- The user should be allowed to upvote a comment only once,
    • And after I add a comment and try to upvote or downvote other comments it doesn't work.
    • You should remove all of the 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
  • Arsh Goyal•940
    @arshGoyalDev

    To make it center you just need to remove this piece of code:

    @media screen and (min-width: 768px){ body { display: block; } }

    As it is making body a block on the devices with min-width of 768px And that is why container is not centered

    And except this you have done a great work.

    Marked as helpful
  • Arsh Goyal•940
    @arshGoyalDev

    Its great but main functionality is not working properly so I think you should use javascript to make it better.

  • Arsh Goyal•940
    @arshGoyalDev

    You have done a great work

  • Arsh Goyal•940
    @arshGoyalDev

    Hello , Its a good solution but it's showing me only 8 at the start and when I filter them it filters the countries from that 8 countries only. I think you need to fix that.

    Marked as helpful
  • Arsh Goyal•940
    @arshGoyalDev

    Great work!

  • Arsh Goyal•940
    @arshGoyalDev

    The explore-btn should point to the destination page but its doing nothing.

    Marked as helpful