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

When downsizing to a smaller MnM container, UI prevents decrementing & editing to smaller quantities #27

Open
limist opened this issue May 16, 2023 · 4 comments

Comments

@limist
Copy link

limist commented May 16, 2023

@helgatheviking When editing a subscription, and after having upsized the subscription to a larger MnM variation/container, there looks to be a UI bug:

  • From the view-subscription page, click the Update Bundle button
  • Select a smaller variation than before
  • To satisfy the lower items count, try clicking any of the -/decrement buttons; the red tooltip flashes e.g. "Container limited to 20" and prevents the decrement from working.
  • Trying to directly edit the quantity field of any item to a smaller or zero quantity, brings up the same red tooltip, and immediately resets to the original value.

Thus it's impossible for the user to downsize their subscription.

@limist
Copy link
Author

limist commented May 16, 2023

@helgatheviking On this same use-case, it would be helpful to have the Mobile Styles footer give some feedback (e.g. instructions, red progress bar) for this over-filled case, to make it clear to the user to lower the items count. Thanks!

@limist
Copy link
Author

limist commented May 16, 2023

Quick update that this problem also appears in the wp-admin backend, for the case when one of our support staff would want to downsize and edit the smaller variation's quantities — can't do so, UI red tooltip blocks all attempts to decrement.

@helgatheviking
Copy link
Contributor

Thanks @limist I think this is a 2 part problem 1. core MNM needs a better way to get out of this situation. That's why mostly for core i've tried to make it impossible to get in to this overfilled state. and 2. I think the JS-prefilling may be related to this... need to try to smooth that out.

@helgatheviking
Copy link
Contributor

@limist I believe we are clearing selections on downsize now to prevent this. Is this still an issue?

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

2 participants