-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Add to cart alert behaviour #1445
Comments
I'll reproduce this behavior and work on solving it over the next couple of days. |
@aaronjudd reducing the slide out delay |
The slideout delay is 4s so that the toast remains on the screen long enough for you to read it. You could probably reduce it some, but making it .4s is way too fast. |
@andela-aatanda agree with Brent, the goal here is a smooth visual effect that is noticeable and gives time for a user to see the action, this issue is not about the behaviour of adding one item, but the effect that occurs while you add more than one item and it's already slide out once. |
Gotcha. |
@zenweasel I was thinking if the |
@zenweasel and I were talking this through and here's what we came up with. When it updates the alert, display a spinner. (you can use the same spinner we use on buttons like the complete payment button). Text should quick fade down, display spinner, and fade back up. See rough mockup. |
@rymorgan This put things in better perspective. Thanks |
Expected behavior
When I rapidly add multiple items to cart, I expect there to be some recurring updates in the "Add to Cart" slideout.
Actual Behavior
Formerly this was implemented using jQuery easing, which provided some queueing, and would display additional updates when additional items were added to the cart. Now using just CSS, we aren't getting additional updates
Steps to Reproduce the Behavior
Add item to cart, see slider. Add to cart again (and again, and again) and slide out will not appear
The text was updated successfully, but these errors were encountered: