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

Merge features #19206

Merged
merged 81 commits into from Oct 15, 2019
Merged

Merge features #19206

merged 81 commits into from Oct 15, 2019

Conversation

afc163 and others added 30 commits September 5, 2019 13:39
* feat(Anchor): add onChange prop

* test(Anchor): add onChange prop

* update snapshots
* feat: upload add download icon

* feat: clear up

* feat: log

* empty

* feat: 理解错误

* feat: test

* test

* feat: clean up

* Update components.test.js.snap

* Update demo.test.js.snap

* Update demo.test.js.snap

* Update demo.test.js.snap

* Update demo.test.js.snap

* Update work-with-us.zh-CN.md

* feat: clean up

* fix: showDownloadIcon=false 24px

* fix: More rigorous

* fix: test

* fix: test

* fix: [] key

* fix: hover css

* fix: test
* feat(Input.Search): add loading prop

* fix: review change

* optimize input search loading prop doc

* fix: ci fail and onSearch disabled when loading
* Add visible to filterDropdown function

* Update tests
chore: merge master into feature
* row vertical gutter

* run test
* add tree-select async demo

* TreeSelect支持clearIcon和removeIcon

* remove docs and demo

* run test

* clear code

* add ui test
@request-info
Copy link

request-info bot commented Oct 14, 2019

We would appreciate it if you could provide us with more info about this issue/pr! Please provide a online reproduction by forking this link https://u.ant.design/codesandbox-repro or a minimal GitHub repository. Issues labeled by Need Reproduce will be closed if no activities in 7 days.

Copy link

@tests-checker tests-checker bot left a comment

Choose a reason for hiding this comment

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

Could you please add tests to make sure this change works as expected?

@codecov
Copy link

codecov bot commented Oct 14, 2019

Codecov Report

Merging #19206 into master will increase coverage by 0.44%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #19206      +/-   ##
==========================================
+ Coverage   97.32%   97.76%   +0.44%     
==========================================
  Files         282      282              
  Lines        7559     7622      +63     
  Branches     2116     2096      -20     
==========================================
+ Hits         7357     7452      +95     
+ Misses        201      170      -31     
+ Partials        1        0       -1
Impacted Files Coverage Δ
components/layout/Sider.tsx 95.6% <ø> (ø) ⬆️
components/config-provider/context.ts 100% <ø> (ø) ⬆️
components/grid/RowContext.tsx 100% <ø> (ø) ⬆️
components/table/Table.tsx 99.48% <100%> (+4.09%) ⬆️
components/config-provider/index.tsx 97.14% <100%> (+0.17%) ⬆️
components/menu/index.tsx 98.31% <100%> (-0.02%) ⬇️
components/page-header/index.tsx 100% <100%> (ø) ⬆️
components/upload/UploadList.tsx 100% <100%> (ø) ⬆️
components/input/Search.tsx 100% <100%> (ø) ⬆️
components/message/index.tsx 94.28% <100%> (+0.43%) ⬆️
... and 12 more

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 f7c37e5...338018e. Read the comment docs.

@netlify
Copy link

netlify bot commented Oct 14, 2019

Deploy preview for ant-design ready!

Built with commit 338018e

https://deploy-preview-19206--ant-design.netlify.com

* refactor: Use CSSMotion

* fix lint

* update rc-menu deps

* fix test case

* clean up

* more test case

* fix lint
@buildsize
Copy link

buildsize bot commented Oct 14, 2019

File name Previous Size New Size Change
package-lock.json 902.75 KB 911.05 KB 8.3 KB (1%)

* pure page header style

* add type props

* add prue demo

* add config provider doc

* fix lint

* fix lint error

* change pure to ghost

* fix test

* default ghost is true

* fix review style

* fix review warning

* use online style

* remove defualt and remove comment

* add test
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.

None yet