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

修复当自定义表格体功能(components.body)和表头分组功能(column.children)同时使用时,会出现表头和表体不对齐的问题 #1003

Merged
merged 2 commits into from
Aug 15, 2023

Conversation

4brother
Copy link
Contributor

@4brother 4brother commented Jul 13, 2023

@vercel
Copy link

vercel bot commented Jul 13, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
table ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 13, 2023 7:35am

};
const wrapper = mount(<Demo columns={columns} data={data} />);
expect(wrapper.find('col').at(noChildColLen + ChildColLen * 2 - 1).props().style.width + wrapper.find('col').last().props().style.width).toEqual(width);
})

Check notice

Code scanning / CodeQL

Semicolon insertion Note test

Avoid automated semicolon insertion (98% of all statements in
the enclosing function
have an explicit semicolon).
@codecov
Copy link

codecov bot commented Jul 13, 2023

Codecov Report

Merging #1003 (89a6307) into master (5fb8f12) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 89a6307 differs from pull request most recent head 752fd87. Consider uploading reports for the commit 752fd87 to get more accurate results

@@           Coverage Diff           @@
##           master    #1003   +/-   ##
=======================================
  Coverage   99.52%   99.52%           
=======================================
  Files          40       40           
  Lines        3563     3563           
  Branches      476      478    +2     
=======================================
  Hits         3546     3546           
  Misses         17       17           
Impacted Files Coverage Δ
src/Table.tsx 99.76% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@zombieJ zombieJ merged commit a3bba58 into react-component:master Aug 15, 2023
7 checks passed
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