@EmmanuelOloke
Posted
Hello Adrian, really awesome job you've done with this challenge I must say, well done!
I really love the mobile view menu implementation, especially that sick transition, awesome job with that.
Just some observations I'd like to highlight; when I clear the cart using the delete button in the navbar cart dropdown, the checkout button still shows, which is contrary to the design file provided. Also, the number of items doesn't change back to 0 once the cart has been cleared. I think you can handle this with React's useState hook.
Another is when I view the website using developer tools, there are some messages that show up in the console as I navigate through the lightbox/modal images. Maybe some leftover console.log debugging messages😅
All in all, really great job you've done. Kudos!!
@Adrian397
Posted
@EmmanuelOloke Thanks a lot. I will correct things u mentioned and yes, i can see now that i didnt get rid of the console logs 😅
@EmmanuelOloke
Posted
@Adrian397 You're welcome. One last little thing I noticed but forgot to mention earlier, on the profile icon in the navbar, you can use CSS outline property like so {outline: 1px solid hsl(26, 100%, 55%);} to set the hover state instead of border. That will prevent the image from being distorted when a user hovers, and still give that surrounding color on hover.
Hope you find it helpful.
Marked as helpful
@Adrian397
Posted
@EmmanuelOloke Completely forgot about it, thanks!
@EmmanuelOloke
Posted
@Adrian397 You're welcome.
