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

Millent修复x-input type =number 在安卓qq浏览器下 v-model 绑定无效 #3083

Closed
wants to merge 2 commits into from
Closed

Conversation

Jon-Millent
Copy link
Contributor

#2960
经过测试发现的原因,是 input.selectionStarttypenumber的时候部分浏览器不允许选择,所以报错触发不了 $emit('input')
39172193254158860.jpg

目前唯一允许安全选择文本的元素是:
如下所述:
参考
https://stackoverflow.com/questions/21177489/selectionstart-selectionend-on-input-type-number-no-longer-allowed-in-chrome

@Jon-Millent Jon-Millent closed this Sep 4, 2018
@codecov-io
Copy link

Codecov Report

Merging #3083 into v2 will increase coverage by 0.33%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##               v2    #3083      +/-   ##
==========================================
+ Coverage   19.45%   19.78%   +0.33%     
==========================================
  Files         226      226              
  Lines        5603     5605       +2     
  Branches      710      710              
==========================================
+ Hits         1090     1109      +19     
+ Misses       4394     4360      -34     
- Partials      119      136      +17
Impacted Files Coverage Δ
src/components/x-input/index.vue 14.7% <100%> (+11.13%) ⬆️
src/components/inline-calendar/util.js 71.76% <0%> (-7.06%) ⬇️
src/components/blur/blur.js 66.66% <0%> (+2.15%) ⬆️
src/components/x-img/index.vue 25% <0%> (+5%) ⬆️
src/libs/base.js 45.45% <0%> (+27.27%) ⬆️

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 690b78b...4bbf705. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants