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

🚀 - Cursor position calculation after keypress & paste #540

Closed
ilterugur opened this issue Sep 29, 2023 · 2 comments
Closed

🚀 - Cursor position calculation after keypress & paste #540

ilterugur opened this issue Sep 29, 2023 · 2 comments
Assignees
Labels
feature New feature or request

Comments

@ilterugur
Copy link

Which package(s) are relevant/related to the feature request?

@maskito/core

Description

Actually, I don't know how can I tell this so I will try to tell this with an example case:

On react-masked-input package, when I use "0 (599) 999 99 99" as a mask pattern, the text input box starts with an empty value and on my first keypress, It automatically calculates if I press "0", "(" or "5" and then it prevents me to put 5 twice on the beginning like "0 (55": instead, it writes down "0 (5".

I checked react-masked-input's codes a little bit and I guess the related code is this: https://github.com/lucasbasquerotto/react-masked-input/blob/master/src/utils/mask.ts#L103

Lots of our users get confused because of this missing cursor position calculation feature and I believe it would be great if you guys add this feature to maskito

@ilterugur ilterugur added the feature New feature or request label Sep 29, 2023
@github-project-automation github-project-automation bot moved this to 💡 Backlog in Taiga-family Sep 29, 2023
@nsbarsukov
Copy link
Member

nsbarsukov commented Oct 2, 2023

@ilterugur Hello!

I make an StackBlitz example for your case:
https://stackblitz.com/edit/maskito-issue-540

It has expected behaviour:

issue-540-demo.mov

Could you check this Stackblitz ?
If your issue still persists or I incorrectly understand your problem, make your Stackblitz reproduction + video-demonstration, please.

Use this starter: https://maskito.dev/stackblitz

@nsbarsukov
Copy link
Member

Closing this issue due to inactivity.
Please feel free to open new issue if you are still running into any problems.

@nsbarsukov nsbarsukov closed this as not planned Won't fix, can't repro, duplicate, stale Oct 9, 2023
@github-project-automation github-project-automation bot moved this from 💡 Backlog to ✅ Done in Taiga-family Oct 9, 2023
@nsbarsukov nsbarsukov self-assigned this Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
No open projects
Status: Done
Development

No branches or pull requests

2 participants