Skip to content
This repository has been archived by the owner on Aug 30, 2018. It is now read-only.

add user feedback on qty update ajax drawer #130

Closed
wants to merge 1 commit into from
Closed

add user feedback on qty update ajax drawer #130

wants to merge 1 commit into from

Conversation

jholl
Copy link

@jholl jholl commented Jun 20, 2014

add scrollTop() if user updates quantity outside of cart drawer when cart drawer is-visible

There is no obvious close button on ajax cart drawer so user naturally scrolls down. user could click Cart link to close the drawer or simply keep scrolling and leave drawer visible. In this case, if user then adds item or updates quantity to cart from 'Add to Cart' button there is no user feedback that the item was added.

If the user checks drawer after page load, leaves open, and then adds a new item on same page, then visible drawer expands a bit pushing down page and exposing a bit of drawer into view at top (probably not intended experience). If it is a quantity change of an item already in cart, then no feedback/movement. There are probably many routes to address this:

  • Add a more natural close button to the cart Drawer so user inclined to close
  • Add a feedback state when Add to Cart is clicked so user knows something happened
  • Add scrollTop() on callback when cart is visible and quantity changes

I figured the easiest was the latter, and this patch addresses that. Please check.

add scrollTop() if user updates quantity outside of cart drawer when cart drawer is-visible
@cshold
Copy link
Contributor

cshold commented Jul 8, 2014

Addressed this with #142. No close button for the cart drawer yet, but adding it to my list of possible additions.

@cshold cshold closed this Jul 8, 2014
@jholl jholl deleted the patch-9 branch July 8, 2014 13:38
@jholl
Copy link
Author

jholl commented Jul 8, 2014

thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants