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] Search: trigger blur after click search button #1448

Merged
merged 4 commits into from Jul 10, 2018
Merged

[bugfix] Search: trigger blur after click search button #1448

merged 4 commits into from Jul 10, 2018

Conversation

snowtreetree
Copy link
Contributor

[bug fix]Search: 使用px2rem,Search问题内容向上偏移。
[new feature]Search: 点击确定,键盘不能收起。

@codecov-io
Copy link

Codecov Report

Merging #1448 into dev will decrease coverage by 0.21%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #1448      +/-   ##
==========================================
- Coverage   84.58%   84.36%   -0.22%     
==========================================
  Files          96       96              
  Lines        1492     1503      +11     
  Branches      296      302       +6     
==========================================
+ Hits         1262     1268       +6     
- Misses        209      212       +3     
- Partials       21       23       +2
Impacted Files Coverage Δ
packages/field/index.vue 94.59% <100%> (+0.3%) ⬆️
packages/radio/index.vue 42.85% <0%> (-7.15%) ⬇️
packages/button/index.vue 66.66% <0%> (ø) ⬆️

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 8c7dc1e...cbbce39. Read the comment docs.

@snowtreetree
Copy link
Contributor Author

CI有个报错,不知道什么原因,但是通过了。😂。

@chenjiahan
Copy link
Member

可以截图解释一下 『点击确定,键盘不能收起』是在什么场景下吗

@snowtreetree
Copy link
Contributor Author

image

感觉点击搜索之后,键盘收起的体验会比较好一点。

@chenjiahan chenjiahan changed the title Snowzhang/fix vant search [bugfix] Search: trigger blur after click search button Jul 10, 2018
@chenjiahan chenjiahan merged commit 91f45ac into youzan:dev Jul 10, 2018
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.

3 participants