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

b-form-input formatter not replacing input value #4695

Closed
NikoFoX opened this issue Jan 29, 2020 · 0 comments · Fixed by #4701, tghelere/CL#13, mariazevedo88/hash-generator-js#33, tghelere/CL#14 or mariazevedo88/hash-generator-js#34
Closed

b-form-input formatter not replacing input value #4695

NikoFoX opened this issue Jan 29, 2020 · 0 comments · Fixed by #4701, tghelere/CL#13, mariazevedo88/hash-generator-js#33, tghelere/CL#14 or mariazevedo88/hash-generator-js#34
Assignees
@jacobmllr95
Labels
Status: Confirmed Type: Bug

Comments

@NikoFoX
Copy link

NikoFoX commented Jan 29, 2020 •
edited
Loading

This bug was also described by closed issue: #3259

Describe the bug

When using b-form-input component with formatter to replace string (e.g. strip any non-numeric character) - it doesn't replace it.

Steps to reproduce the bug

If you type anything other than number, it will be typed into the input anyway.
If you again type anything other than number (but not the same char) - it will replace the last char.
If you type the same 3 non-numeric chars - 2 of them will stay.

Expected behavior

It should't allow chars to be put into the input (should be stripped instead).

Versions

Libraries:

  • BootstrapVue: 2.3.0

Demo link

https://codepen.io/qElectro/pen/jOEoZGx
@jacobmllr95 jacobmllr95 self-assigned this Jan 30, 2020
@jacobmllr95 jacobmllr95 mentioned this issue Jan 30, 2020
Merged
20 tasks
@tmorehouse tmorehouse mentioned this issue Jan 30, 2020
Merged
jacobmllr95 added a commit that referenced this issue Jan 30, 2020
@jacobmllr95 @tmorehouse
fix(b-form-input, b-form-textarea): properly handle out-of-sync values (
954176d 
closes #4695) (#4701)

* fix(form-input/form-textarea): properly handle out-of-sync values

* Update form-text.js

* Update form-text.js

Co-authored-by: Troy Morehouse <[email protected]>
This was referenced Feb 24, 2020
Open
Merged
Closed
Merged
This was referenced Mar 4, 2020
Open
Merged
Merged
@snyk-bot snyk-bot mentioned this issue Apr 7, 2020
Open
This was referenced Apr 15, 2020
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jacobmllr95 jacobmllr95

Labels
Status: Confirmed Type: Bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(b-form-input, b-form-textarea): properly handle out-of-sync values (closes #4695) bootstrap-vue/bootstrap-vue
[Snyk] Upgrade bootstrap-vue from 2.3.0 to 2.4.0 tghelere/CL
[Snyk] Upgrade bootstrap-vue from 2.3.0 to 2.4.0 mariazevedo88/hash-generator-js
[Snyk] Upgrade bootstrap-vue from 2.4.0 to 2.4.1 tghelere/CL
[Snyk] Upgrade bootstrap-vue from 2.4.0 to 2.4.1 mariazevedo88/hash-generator-js
[Snyk] Upgrade bootstrap-vue from 2.3.0 to 2.4.0 michaelwybraniec/vue-4-peaks
[Snyk] Upgrade bootstrap-vue from 2.0.4 to 2.4.0 estafette/estafette-ci-web
[Snyk] Upgrade bootstrap-vue from 2.3.0 to 2.7.0 JuroOravec/powerjinja
[Snyk] Upgrade bootstrap-vue from 2.0.0-rc.24 to 2.8.0 mc-ireiser/coreui-free-vue-admin-template
[Snyk] Upgrade bootstrap-vue from 2.0.0-rc.27 to 2.8.0 mc-ireiser/termodaq_app
3 participants
@jacobmllr95 @tmorehouse @NikoFoX