-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cart #1
Comments
@lizkaraffa Thumbs up on all the opportunity points you've made. I'll update those as I have time next week. Regarding the clarification point, my thought with the edit button was that it would load the product page again so you could choose a different color/size/etc. Is that a valid thought? or should we ditch that button all together? |
@Circa75 regarding the pencil button. Hmm I could see it be awesome or redundant since normally you can click on the product title to go back to the product page. Just realizing the title is not linked up, that'd probably be good to add. Also, functionality might be tricky for the pencil icon, because if you go back to the product page and change variations etc and then click add to cart, it's going to add a new product to your cart and not "edit" the one you already had in there. Maybe if there was an elegant way to change variations while still in the cart? Otherwise it could get ditched. |
Sounds good... I'll think that one over. |
Loves:
Opportunities:
[update]
Clarification:
The text was updated successfully, but these errors were encountered: