Skip to content
This repository has been archived by the owner on Nov 14, 2023. It is now read-only.

feat: VirtualSelectBox multi select #75

Merged
merged 4 commits into from
Sep 23, 2019

Conversation

wangkailang
Copy link
Member

No description provided.

@netlify
Copy link

netlify bot commented Sep 11, 2019

Deploy preview for wizard-ui ready!

Built with commit caf5d61

https://deploy-preview-75--wizard-ui.netlify.com

@codecov
Copy link

codecov bot commented Sep 11, 2019

Codecov Report

Merging #75 into master will decrease coverage by 1.61%.
The diff coverage is 59.74%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #75      +/-   ##
==========================================
- Coverage   82.13%   80.51%   -1.62%     
==========================================
  Files          37       38       +1     
  Lines        1304     1350      +46     
  Branches      261      273      +12     
==========================================
+ Hits         1071     1087      +16     
- Misses        230      260      +30     
  Partials        3        3
Impacted Files Coverage Δ
...rc/components/VirtualSelectBox/SelectCheckItem.tsx 50% <50%> (ø)
src/components/VirtualSelectBox/index.tsx 67.28% <61.19%> (-10.5%) ⬇️

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 0efa43e...caf5d61. Read the comment docs.

…at/VirtualSelectBox-multi-select

* 'master' of https://github.com/xsky-fe/wizard-ui:
  Fix generics error in tsx. (#84)
  Feat/notification list (#73)
  refactor: Update docs code (#83)
  remove docs yarn.lock
  Refactor/action use yarn (#77)
  feat: Github actions (#76)

# Conflicts:
#	docs/content/components/virtual-select-box.mdx
#	docs/gatsby-config.js
#	src/interface.tsx
option = {
// @ts-ignore
label: item.name,
// @ts-ignore
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

如果不想验证 option 类型,可以在定义时显式声明为 any 类型:let option: any;

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里是忽略 item(T 泛型)的类型定义

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

之前 T 有默认类型定义倒是可以处理,现在没有默认的

@lijianxin1202 lijianxin1202 added the 4-Approved Code PR approved label Sep 23, 2019
@wangkailang wangkailang merged commit e30560e into master Sep 23, 2019
@wangkailang wangkailang deleted the feat/VirtualSelectBox-multi-select branch September 23, 2019 04:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
1-Review Code review 4-Approved Code PR approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants