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

fix(Select): fix Select with useVirtual when empty dataSource #164

Merged
merged 2 commits into from
Jan 9, 2019

Conversation

bojoyzhou
Copy link
Contributor

Select 组件开启virtual和showSearch之后,当filter出空dataSource时抛异常。
简单判断children.length为0,关闭virtual

@codecov
Copy link

codecov bot commented Jan 7, 2019

Codecov Report

Merging #164 into master will increase coverage by 0.12%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #164      +/-   ##
==========================================
+ Coverage   88.02%   88.15%   +0.12%     
==========================================
  Files         253      253              
  Lines       12370    12372       +2     
  Branches     3211     3263      +52     
==========================================
+ Hits        10889    10906      +17     
+ Misses       1474     1460      -14     
+ Partials        7        6       -1
Impacted Files Coverage Δ
src/select/base.jsx 89.59% <100%> (+3.46%) ⬆️
src/cascader/cascader.jsx 97.8% <0%> (+0.01%) ⬆️
src/select/select.jsx 89.74% <0%> (+0.36%) ⬆️
src/virtual-list/virtual-list.jsx 94.9% <0%> (+3.7%) ⬆️

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 9498ad9...1e76c8c. Read the comment docs.

@youluna youluna requested a review from bindoon January 8, 2019 02:07
@youluna youluna assigned bindoon and youluna and unassigned youluna Jan 8, 2019
@youluna youluna self-requested a review January 8, 2019 02:08
@youluna
Copy link
Member

youluna commented Jan 8, 2019

It looks all right.

@youluna youluna merged commit 54da63f into alibaba-fusion:master Jan 9, 2019
@bojoyzhou bojoyzhou deleted the select branch January 9, 2019 07:40
youluna pushed a commit that referenced this pull request Jan 15, 2019
* fix(Select): fix Select with useVirtual when empty dataSource
tao1991123 added a commit that referenced this pull request Jan 16, 2019
* upstream-master:
  test(*): ignore compatible with 0.x
  fix(Slider): can swipe vertical & width 100% invertical
  fix(Nav): overflow when icon-only without icon
  test(Table): edit test cases
  fix(Tab): extra align middle by default
  fix(Loading): fix fullscreen in demo - add button as safe node to disable document event handler
  chore(*): resest version of webpack-dev-server
  chore(*): Release-1.11.8
  docs(ConfigProvider): add docs
  fix(Nav): icon-only overflow bug fix
  fix(Radio): button shape hover border covers checked border
  fix(Form): remove warning while message write in jsx
  fix(Form): switch aligin while size=large
  fix(Select): fix Select with useVirtual when empty dataSource (#164)
  fix(Validate): value can be false
  fix(Nav): can't config nav group label font size/height
  Update License
  fix(CascaderSelect): value type not equal dataSource value
  fix(Timeline): timline-item warning  has function type prop

# Conflicts:
#	src/nav/main.scss
youluna pushed a commit that referenced this pull request Jan 23, 2019
* fix(Select): fix Select with useVirtual when empty dataSource
youluna added a commit that referenced this pull request Jan 23, 2019
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