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

rowSelection.columnWidth is not respected #16163

Closed
1 task done
hon2a opened this issue Apr 18, 2019 · 4 comments
Closed
1 task done

rowSelection.columnWidth is not respected #16163

hon2a opened this issue Apr 18, 2019 · 4 comments

Comments

@hon2a
Copy link

@hon2a hon2a commented Apr 18, 2019

  • I have searched the issues of this repository and believe that this is not a duplicate.

Reproduction link

Edit on CodeSandbox

Steps to reproduce

Create a table that has a row selection column with custom width (<Table rowSelection={{ columnWidth: 40, ... }} ... />).

What is expected?

The width of the row selection column should be 40px, as specified by rowSelection.columnWidth.

What is actually happening?

The width of the column is the default 60px, not the specified custom width.

Environment Info
antd 3.16.3
React 16.9.0
System MacOS 10.14.4 (18E226)
Browser Chrome 73.0.3683.103 (Official Build) (64-bit)

Please note that rowSelection.columnWidth should be applied even when the row selection column is fixed.

@zombieJ
Copy link
Member

@zombieJ zombieJ commented Apr 18, 2019

@afc163 Seems this change breaks the columnWidth: d922c37#diff-e2a536684ed8b2684e0ae68f0ef1ffe9R319

Loading

@hon2a
Copy link
Author

@hon2a hon2a commented Apr 18, 2019

I've also looked at the changelog and expected this to be the case, but I don't see how that patch could possibly do it.

Loading

@zombieJ
Copy link
Member

@zombieJ zombieJ commented Apr 19, 2019

@hon2a , for current path you can use css to set the column width. Let me handle it in antd side.

Loading

@hon2a
Copy link
Author

@hon2a hon2a commented Apr 19, 2019

@zombieJ Thanks for the tip; I've already added a styling override for the default width in our projects.

Loading

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants