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

Update Cart with items display #2707

Closed
kirstenalarsen opened this issue Sep 13, 2018 · 3 comments
Closed

Update Cart with items display #2707

kirstenalarsen opened this issue Sep 13, 2018 · 3 comments

Comments

@kirstenalarsen
Copy link
Contributor

kirstenalarsen commented Sep 13, 2018

image

Zeplin: https://zpl.io/bAEMxv8

Based on current shopping cart functionality:

  • Items between OFN header and checkout + total footer scrolls
  • User can tap ‘cart’ in header to close cart
  • Removed explicit ‘edit cart’ button from the design - @kirsten please let me know if you think this function is a must have.
  • $XX is the total value for each product listing, removing the 1 x $100.00 = $100.00 version of product pricing
  • Cart total + checkout button is fixed to bottom of viewport - [mobile ux] Sticky footer element on Cart dropdown #2718
@Matt-Yorkley
Copy link
Contributor

On desktop there's buttons for both edit card and checkout. There's no buttons here for edit cart. I'd say it's pretty essential for mobile UX...

@Matt-Yorkley Matt-Yorkley self-assigned this Aug 5, 2019
@kirstenalarsen
Copy link
Contributor Author

this is the problem with the timelapse between design and dev . . I am pretty sure we discussed this but I cannot remember how we came to this decision. There is a google slide pack that has lots of the discussion and sometimes more info than zeplin, have a look at slide 16 . . where Yuko says she removed it and I didn't respond saying put it back!

Would a text link for Edit Cart in the top menu, between '54 Items in your cart' and 'Close' do the trick?

I will post a message in slack to start discussion of if/how we reengage Yuko to work with us as we're hitting need for adjustments . .

@daniellemoorhead
Copy link
Contributor

Duplicate of #4704

@daniellemoorhead daniellemoorhead marked this as a duplicate of #4704 Jan 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants