Skip to content
This repository has been archived by the owner on Mar 2, 2022. It is now read-only.

🐛 bug: fix toolbar second search doesn't work #647

Merged
merged 1 commit into from
Jul 22, 2020
Merged

🐛 bug: fix toolbar second search doesn't work #647

merged 1 commit into from
Jul 22, 2020

Conversation

whatthehan
Copy link
Contributor

修复 #645

修复 toolbar 二次搜索不生效问题
example示例修改keyword

@codecov-commenter
Copy link

Codecov Report

Merging #647 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #647   +/-   ##
=======================================
  Coverage   17.15%   17.15%           
=======================================
  Files          21       21           
  Lines        1154     1154           
  Branches      379      379           
=======================================
  Hits          198      198           
  Misses        956      956           
Impacted Files Coverage Δ
src/Table.tsx 20.00% <ø> (ø)

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 57ec2fe...e834653. Read the comment docs.

@@ -913,8 +913,8 @@ const ProTable = <T extends {}, U extends object>(
}
: options.search;
setFormSearch({
[name]: keyword,
...formSearch,
Copy link
Contributor

Choose a reason for hiding this comment

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

formSearch 里面可能有 keyword,是不是做一下处理?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

是指表单中的name和toolbar的name重名的情况吗...有点没看懂😂

Copy link
Contributor

Choose a reason for hiding this comment

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

用户自己配置的,其实可以不用管

@chenshuai2144 chenshuai2144 merged commit 4b1307a into ant-design:master Jul 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants