Skip to content

/ fakerpress

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

Jump to bottom

Can't select quantity using selector - FakerPress #124

Closed
rahmantanvir opened this issue Feb 3, 2018 · 1 comment
Closed

Can't select quantity using selector - FakerPress #124

rahmantanvir opened this issue Feb 3, 2018 · 1 comment
Assignees
@bordoni
Labels
status:confirmed type:bug
Milestone
Version: 0.4.10

Comments

@rahmantanvir
Copy link

@rahmantanvir rahmantanvir commented Feb 3, 2018 •
edited

Expected behavior

Quantity should be changeable using selector.

Actual behavior

Quantity is vanishing or not properly changing in the plugin.

How to reproduce

  1. Install WordPress.
  2. Install FakerPress plugin and activate it.
  3. Give a random amount of quantity.
  4. Try to change min quantity using selector.
  • Browser: Chrome
  • Operating system: Windows 10

Recording Of The Bug

bug1.gif
@bordoni bordoni added this to the Version: 0.4.10 milestone Feb 6, 2018
@bordoni bordoni self-assigned this Feb 6, 2018
@bordoni
Copy link
Owner

@bordoni bordoni commented Feb 6, 2018

Ohhh nice catch, I will take a look at it.
bordoni added a commit that referenced this issue Feb 11, 2018
@bordoni
Fix Quantity fields problems with JS
Verified
This commit was signed with a verified signature.
bordoni Gustavo Bordoni
GPG key ID: D9EC8495E9D1F6B8 Learn about signing commits
0cc1de0 
* Resolve #124
* Resolve #90
@bordoni bordoni mentioned this issue Feb 11, 2018
Merged
@bordoni bordoni closed this in #125 Feb 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bordoni bordoni

Labels
status:confirmed type:bug
Projects
None yet
Milestone
Version: 0.4.10
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@bordoni @rahmantanvir