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

Number Input increment / decrement sometimes require 2 clicks at min / max respectively. #4776

Closed
1 of 3 tasks
rafaelderolez opened this issue Oct 1, 2021 · 4 comments
Closed
1 of 3 tasks
Labels
needs triage Issues and pull requests that need triage attention stale Type: Bug 🐛 Something isn't working

Comments

@rafaelderolez
Copy link

Description

With the mobile spinner (Number Input) example from the docs, when the spinner is at its minimum it sometimes requires two clicks on the increment button to actually do anything (and vice-versa to decrement at max value)

Link to Reproduction

https://codesandbox.io/s/chakra-ui-number-input-inc-dec-bug-dfuq2

Steps to reproduce

  1. Visit Sandbox and open console
  2. Click minus button to decrement to minimum value of 1
  3. Click on plus button to increment
  4. Expect value to increase to 2 but remains at 1
  5. Click plus button again to increment
  6. Value finally increases to 2

Chakra UI Version

1.6.8

Browser

Brave V1.30.86

Operating System

  • macOS
  • Windows
  • Linux

Additional Information

Kapture 2021-10-01 at 14 03 39

@rafaelderolez rafaelderolez added needs triage Issues and pull requests that need triage attention Type: Bug 🐛 Something isn't working labels Oct 1, 2021
@xotea
Copy link

xotea commented Oct 1, 2021

#4646 might be related. If i copy your code and paste it to the codesandbox of my PR (#4703) the issue seems to be gone.

@stale
Copy link

stale bot commented Nov 9, 2021

Hi!
This issue has been automatically marked as stale because lack of recent activity. It will be closed if no further activity occurs within 5 days. Thank you for your contributions.

@stale stale bot added the stale label Nov 9, 2021
@anubra266
Copy link
Collaborator

will be fixed in #4982

@samhecquet-divido
Copy link

Hi @anubra266 , any update on #4982 being merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs triage Issues and pull requests that need triage attention stale Type: Bug 🐛 Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants