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

[NInputNumber] Strange loop when @update:value #3329

Closed
vicenterusso opened this issue Jul 22, 2022 · 4 comments
Closed

[NInputNumber] Strange loop when @update:value #3329

vicenterusso opened this issue Jul 22, 2022 · 4 comments
Labels
bug Something isn't working

Comments

@vicenterusso
Copy link
Contributor

TuSimple/naive-ui version (版本)

2.25.3

Vue version (Vue 版本)

3.2.37

Browser and its version (浏览器及其版本)

Tested on Chrome and Firefox

System and its version (系统及其版本)

Linux Fedora 36

Node version (Node 版本)

Reappearance link (重现链接)

https://codesandbox.io/s/gallant-bassi-d5cr5i?file=/src/Demo.vue

Reappearance steps (重现步骤)

Source:
https://codesandbox.io/s/gallant-bassi-d5cr5i?file=/src/Demo.vue

To reproduce:

  1. Open link [ https://d5cr5i.csb.app/ ]
  2. Open devtools (sometimes it doesnt work with dev tools not present)
  3. Fast click to increase or decrease
  4. Loop starts

Video:
https://i.imgur.com/6RU9Vto.mp4

Expected results (期望的结果)

Expected no loop behaviour

Actual results (实际的结果)

Loop starts when fast click

Remarks (补充说明)

Bug happens from version 2.25.3 to the latest.

@github-actions github-actions bot added the untriaged need to sort label Jul 22, 2022
@07akioni 07akioni added bug Something isn't working and removed untriaged need to sort labels Jul 24, 2022
@07akioni
Copy link
Collaborator

07akioni commented Jul 24, 2022

I can't reproduce it but it may be relevant with
#3333

Maybe you can change node_modules's code and see if the issue is still there.

If it's fixed, that patch would be released in next version.

@OrbisK
Copy link
Collaborator

OrbisK commented Jul 28, 2022

I managed to repod it, with a macro that presses 30 times (every 2ms).

@vicenterusso
Copy link
Contributor Author

I confirm it's not fixed, version 2.32.1

@vicenterusso
Copy link
Contributor Author

Confirmed: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants