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

bugfix(int-512): fix mask in sbTextField and SbDatepicker #296

Merged
merged 9 commits into from
Aug 31, 2022

Conversation

fernandacanto
Copy link
Contributor

In this PR we had to remove the v-mask package, because the focus was getting lost, this issue is already mapped and there are 2 issues open in the v-mask package (probil/v-mask#59 and probil/v-mask#465), but as we can't wait for the fix I had to change the package to maska.

I recorded this video showing the problem happening:

Gravacao.de.Tela.2022-08-29.as.16.16.03.mov

Pull request type

Jira Link: INT-512

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Other (please describe):

How to test this PR

Watch the video and check if the errors shown are no longer happening.

@vercel
Copy link

vercel bot commented Aug 29, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
storyblok-design-system ✅ Ready (Inspect) Visit Preview Aug 30, 2022 at 6:07PM (UTC)

@carlosericciardi
Copy link

The date is applied only if using "enter", it is good to persist the date changed on the focus lost too.

@thiagosaife thiagosaife merged commit b9057a1 into develop Aug 31, 2022
@thiagosaife thiagosaife deleted the bugfix/int-512-fix-mask branch August 31, 2022 11:24
@github-actions
Copy link

🎉 This PR is included in version 1.12.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
easy to review This PR is easy to review released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants