e-commerce product with CSS Flex.

@AhmadYousif89 Thank you for the feedback. I have made changes to the webpage but I can't understand what you mean by "I can open the cart modal" (point 5). Have you some time to take another look at the page?? I will be happy for any other changes to the page

@NermenElefky

I see you have fixed a lot of the issues you had before 👏 good job, but I still see other issues :

• you added the ability to click "anywhere" to close the cart modal, but it wasn't a clean fix but more like a bandied fix, because if I click on the top header it doesn't close unless I click outside the header, instead you should detect if a user is not clicking on the element itself and then close the cart otherwise " clicking inside/on the cart icon " remains open or reopen it.

• clicking on the backdrop does close it but so does the side menu., meaning that if I click inside the backdrop or anywhere inside the side menu it also closes the menu, which is not the ideal behavior user should only close the menu if he/she clicks explicitly on the backdrop or the close icon on the top left corner.

• the side menu sometimes acts funny, if I click on the close icon or one of the links it closes the menu as I mentioned in the previous issue but now if I click again to open it I see another overlay on top of the menu and looks distorted and the link gets shifted up to the top left corner 🤷‍♂️ see here

• using img elements as buttons is not a good idea.

• I see that you are wrapping the elements with the backdrop overlay as a container and that's not good the backdrop should only serve one purpose which is displaying a " backdrop on demand " so it shouldn't really take any children inside of it or have the need to know about them (unless of course the business logic needs it somehow).

if you have any questions don't hesitate to ask 👍

@AhmadYousif89

Thanks for your feedback 🙂. It was very useful and I learned a lot. Have you some time to take another look at the page ?? I am very interested about your next feedback😊.

@NermenElefky

Hi again 👋

I'm so glad that you have fixed all of the previous issues you had before, I have some minor suggestions they are mainly around styling :

• the cart icon size looks good on the mobile view but on the desktop, it's a bit small.
• the shape of the counter on the cart icon doesn't look good maybe adding some inline padding would fix that.
• the main image has a cursor pointer on it for no reason indicating that it's clickable where it's not (in mobile).
• the discount badge is a bit small and needs to be bold so as the original price IMO.
• there is no visual feedback after clicking the "add to cart" button indicating an action has occurred (this is maybe the most important out of all).

overall you have done a great job 👍 keep up the good work.

btw I just updated my solution with new features let me hear your thoughts about it 😃

@AhmadYousif89

Thank you for taking the time to help me in the project😊. Your feedback was very helpful.