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

feat: Add InputDropdown. #64

Merged
merged 8 commits into from
Aug 30, 2019
Merged

feat: Add InputDropdown. #64

merged 8 commits into from
Aug 30, 2019

Conversation

lijianxin1202
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented Aug 23, 2019

Deploy preview for wizard-ui ready!

Built with commit 6f9ce30

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

@codecov
Copy link

codecov bot commented Aug 23, 2019

Codecov Report

Merging #64 into master will increase coverage by 1.12%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #64      +/-   ##
==========================================
+ Coverage   76.94%   78.07%   +1.12%     
==========================================
  Files          29       30       +1     
  Lines         937      967      +30     
  Branches      195      201       +6     
==========================================
+ Hits          721      755      +34     
+ Misses        214      210       -4     
  Partials        2        2
Impacted Files Coverage Δ
src/components/InputDropdown/index.tsx 100% <100%> (ø)
src/components/index.tsx 100% <100%> (ø) ⬆️
src/utils/uuid.ts 100% <0%> (+80%) ⬆️

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 60e361d...6f9ce30. Read the comment docs.

src/components/InputDropdown/index.tsx Outdated Show resolved Hide resolved
if (input && input.value !== undefined) {
return input.value;
}
return value !== undefined ? value : defaultValue;
Copy link
Member

Choose a reason for hiding this comment

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

return value || defaultValue

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这么写和原逻辑就不一致了,value 为 0,空字符串都会返回 defaultValue 了。

@wangkailang
Copy link
Member

image
高度不一致

@lijianxin1202
Copy link
Contributor Author

高度不一致是由于 sass 精度导致的,修改对应 loader config 就好了

@lijianxin1202 lijianxin1202 added the 3-Changed Code has Changed label Aug 28, 2019
@wangkailang wangkailang added the 4-Approved Code PR approved label Aug 30, 2019
@wangkailang wangkailang merged commit a1dcf2a into master Aug 30, 2019
@wangkailang wangkailang deleted the feat/inputdropdown branch August 30, 2019 10:26
wangkailang added a commit that referenced this pull request Sep 2, 2019
…al-select-box

* 'master' of github.com:xsky-fe/wizard-ui:
  Release 0.2.0
  feat: Add InputDropdown. (#64)

# Conflicts:
#	src/interface.tsx
wangkailang added a commit that referenced this pull request Sep 2, 2019
…odal-props-hideFooter

* 'master' of github.com:xsky-fe/wizard-ui:
  feat: VirtualSelectBox (#65)
  Release 0.2.0
  feat: Add InputDropdown. (#64)

# Conflicts:
#	src/components/VirtualList/index.tsx
#	src/components/index.tsx
#	src/interface.tsx
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
1-Review Code review 3-Changed Code has Changed 4-Approved Code PR approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants