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.

Sign up for GitHub

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

🐞 - Cursor freeze in case delete digits by DELETE button #3664

Closed
2 of 9 tasks
MarenkovIgor opened this issue Feb 13, 2023 · 1 comment · Fixed by #4725
Closed
2 of 9 tasks

🐞 - Cursor freeze in case delete digits by DELETE button #3664

MarenkovIgor opened this issue Feb 13, 2023 · 1 comment · Fixed by #4725
Assignees
@nsbarsukov
Labels
P2 This issue has medium priority S3 This issue has low severity

Comments

@MarenkovIgor
Copy link

Which @taiga-ui/* package(s) are the source of the bug?

core, kit

Please provide a link to a minimal reproduction of the bug

https://stackblitz.com/edit/angular-tbtqsy?file=src/app/app.component.ts

Is this issue blocking you?

Non-Blocking

Description

inputNumber интересно себя ведёт при удалении цифр кнопкой delete: delete не удаляет с переходом между группами разрядов. Пример удаляя от 1 000 000 нули, курсор и кнопка останавливается на пробеле не переходя в следующую группу разрядов.

Steps to reproduce:

  1. Open https://stackblitz.com/edit/angular-tbtqsy?file=src/app/app.component.ts
  2. Place cursor here: 1 |000 000
  3. Press DELETE button (windows) 3 times, now you see 1| 000
  4. Press DELETE again
  5. Nothing happend

Angular version

12.2.17

Taiga UI version

3.17.0

Which browsers have you used?

  • Chrome
  • Firefox
  • Safari
  • Edge

Which operating systems have you used?

  • macOS
  • Windows
  • Linux
  • iOS
  • Android
@nsbarsukov nsbarsukov added scope: mask P3 This issue has low priority labels Feb 13, 2023
@splincode splincode added scope: kit and removed P3 This issue has low priority labels Feb 13, 2023
@nsbarsukov nsbarsukov added P3 This issue has low priority and removed state: need triage labels Feb 13, 2023
@splincode splincode added scope: kit P3 This issue has low priority and removed P3 This issue has low priority labels Feb 13, 2023
@splincode splincode added P2 This issue has medium priority and removed P3 This issue has low priority labels Mar 7, 2023
@Reactiver
Copy link
Contributor

Reactiver commented Mar 7, 2023 •
edited
Loading

Got the same problem on mobile(iOS). For desktop works well.

InputNumber with decimal="always"

2023-03-07.12.36.06.mov

@waterplea waterplea added S3 This issue has low severity and removed scope: kit labels Mar 31, 2023
nsbarsukov added a commit that referenced this issue Jun 20, 2023
@nsbarsukov
chore(demo-integrations): new Cypress tests for #3664 bug
28b6697
@nsbarsukov nsbarsukov mentioned this issue Jun 20, 2023
Merged
6 tasks
@nsbarsukov nsbarsukov self-assigned this Jun 20, 2023
nsbarsukov added a commit that referenced this issue Jun 20, 2023
@nsbarsukov
chore(demo-integrations): new Cypress tests for #3664 bug
8e89815
nsbarsukov added a commit that referenced this issue Jun 23, 2023
@nsbarsukov
chore(demo-integrations): new Cypress tests for #3664 bug
60fbf03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nsbarsukov nsbarsukov

Labels
P2 This issue has medium priority S3 This issue has low severity
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(kit): InputNumber uses Maskito instead of legacy text-mask taiga-family/taiga-ui
5 participants
@waterplea @splincode @MarenkovIgor @nsbarsukov @Reactiver