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

🐞 - maskitoWithPlaceholder with focusedOnly false not visible #609

Closed
2 of 9 tasks
ambrodrigues opened this issue Oct 18, 2023 · 2 comments
Closed
2 of 9 tasks
Assignees
Labels
bug Something isn't working

Comments

@ambrodrigues
Copy link

Which package(s) are the source of the bug?

@maskito/kit

Playground Link

https://stackblitz.com/edit/ifye36?file=index.ts,mask.ts

Description

when using the maskitoWithPlaceholder method with the parameter focusedOnly = false the placeholder is not visible. I believe that the placeholder should always appear with this false parameter

Maskito version

1.8.0

Which browsers have you used?

  • Chrome
  • Firefox
  • Safari
  • Edge

Which operating systems have you used?

  • macOS
  • Windows
  • Linux
  • iOS
  • Android
@ambrodrigues ambrodrigues added the bug Something isn't working label Oct 18, 2023
@github-project-automation github-project-automation bot moved this to 💡 Backlog in Taiga-family Oct 18, 2023
@ambrodrigues ambrodrigues changed the title 🐞 - 🐞 - maskitoWithPlaceholder with focusedOnly false not visible Oct 18, 2023
@nsbarsukov
Copy link
Member

@ambrodrigues Hello!

Just initialize input-element with right value.
For initially "empty"-date use the following lines:

<input value="dd/mm/yyyy" />

And then everything should works fine.
See this stackblitz: https://stackblitz.com/edit/maskito-date-placeholder-always?file=mask.ts,index.html


Why so ? Why we can't just do it by ourself inside mask?
We need to provide flexibility about this case: some developer want textfield to be initially filled with placeholder, but others don't.
The current behaviour allows to choose.

In Maskito 2.0 we want to change this behaviour a little bit:
#604
Initial calibrations will be optional and user can enable it by maskitoInitialCalibrationPlugin.

@nsbarsukov
Copy link
Member

I will close this issue because further improvements of maskitoWithPlaceholder will be done in this issue:

@nsbarsukov nsbarsukov closed this as not planned Won't fix, can't repro, duplicate, stale Oct 19, 2023
@github-project-automation github-project-automation bot moved this from 💡 Backlog to ✅ Done in Taiga-family Oct 19, 2023
@nsbarsukov nsbarsukov self-assigned this Oct 19, 2023
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
No open projects
Status: Done
Development

No branches or pull requests

2 participants