Skip to content

fix(form-input): bug fixes and add new features #2100

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

Merged
merged 88 commits into from
Oct 27, 2018
Merged

Conversation

tmorehouse
Copy link
Member

@tmorehouse tmorehouse commented Oct 25, 2018

Changes should address some of the issues reported with b-form-input:

fixes #2061, fixes #1988, fixes #1951, fixes #1881, fixes #1863, fixes #1855, fixes #1852, #fixes #1642

May address #2098, #1908, #1518, #2104 , #2131, #2124

Adds new no-wheel prop to disable mouse wheel incrementing number inputs. (fixes #1872)

Adds new mixins for exposing native <input> selection (fixes #2026), range and validity on the component. Also adds focus and blur methods.

Additional tests added.

Similar changes could be made with b-form-textarea once this PR is merged.

@codecov
Copy link

codecov bot commented Oct 25, 2018

Codecov Report

Merging #2100 into dev will decrease coverage by 0.02%.
The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #2100      +/-   ##
==========================================
- Coverage   64.61%   64.58%   -0.03%     
==========================================
  Files         154      156       +2     
  Lines        2922     2948      +26     
  Branches      805      809       +4     
==========================================
+ Hits         1888     1904      +16     
- Misses        744      755      +11     
+ Partials      290      289       -1
Impacted Files Coverage Δ
src/mixins/form-selection.js 0% <0%> (ø)
src/mixins/form-validity.js 0% <0%> (ø)
src/components/form-input/form-input.js 100% <100%> (+20%) ⬆️
src/mixins/form-size.js 50% <0%> (-50%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7a82a6a...43754ba. Read the comment docs.

@tmorehouse tmorehouse changed the title [WIP] fix(form-input): bug fixes [WIP] fix(form-input): bug fixes and new no-wheel prop Oct 25, 2018
@tmorehouse tmorehouse changed the title [WIP] fix(form-input): bug fixes and new no-wheel prop fix(form-input): bug fixes and add new no-wheel prop Oct 25, 2018
@tmorehouse tmorehouse requested a review from pi0 October 25, 2018 16:31
@tmorehouse tmorehouse changed the title [WIP] fix(form-input): bug fixes and add new features fix(form-input): bug fixes and add new features Oct 27, 2018
@tmorehouse tmorehouse merged commit 0299159 into dev Oct 27, 2018
@tmorehouse tmorehouse deleted the tmorehouse/b-form-input branch October 27, 2018 19:21
@pi0
Copy link
Member

pi0 commented Oct 27, 2018

This is GREAT @tmorehouse ❤️

@tmorehouse
Copy link
Member Author

Took a while to get used to the new Vue Test Utils, but I like it.

@avrezvanov
Copy link

Hello. Is this fix in bootstrap-vue@2.0.0-rc.11?

@tmorehouse
Copy link
Member Author

It is currently only available in the dev branch. We hope to have a new version of Bootstrap-Vue released in the near future,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment