Skip to content
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 layout #134

Open
wjcPS opened this issue Apr 10, 2019 · 3 comments · May be fixed by #270
Open

Cart layout #134

wjcPS opened this issue Apr 10, 2019 · 3 comments · May be fixed by #270
Assignees
Labels
Epic: Cart Cart Page

Comments

@wjcPS
Copy link
Collaborator

wjcPS commented Apr 10, 2019

No description provided.

@wjcPS wjcPS added the Epic: Cart Cart Page label Apr 10, 2019
@wjcPS
Copy link
Collaborator Author

wjcPS commented Apr 10, 2019

  • remove and edit
    -- Copy functionality from mini-cart (remove/edit) ? (remove/edit functionality have already been built for mini cart)
    -- edit is just a link out to the original product page
  • Add delete/edit icons to SVG “sprite sheet” ?
  • quantity toggle: quantity is more of an ‘update’

@wjcPS wjcPS self-assigned this Apr 17, 2019
@aevanson aevanson added this to the Global Accessibility Day milestone Apr 18, 2019
@wjcPS wjcPS assigned Puneetsehgal and unassigned wjcPS Feb 27, 2020
@wjcPS
Copy link
Collaborator Author

wjcPS commented Feb 27, 2020

Initial stab at the Cart page: https://github.com/PublicisSapient/accessible-ecommerce-demo/tree/feature/134-cart-layout

Added a screen shot / comp from when we were doing initial work

@wjcPS
Copy link
Collaborator Author

wjcPS commented Feb 27, 2020

CartDesktop

Puneetsehgal added a commit that referenced this issue Mar 6, 2020
Update the Cart layout to match the comp and implement the quantity change and remove item features
@Puneetsehgal Puneetsehgal linked a pull request Mar 6, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Epic: Cart Cart Page
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants