Column header row not resizing as expected

[m-preiner]

Expected Behavior

When there are long column names, the header row height is expected to adjust to fit the entire wrapped column name.

Current Behavior

The column header cells are not resizing their height when the text begins to wrap. This causes the column headers to overlap with the first row.

Steps to Reproduce (for bugs)

I've posted a codesandbox example at :
https://codesandbox.io/embed/frosty-beaver-kcwnt.
Note: you'll probably need to open in the editor and then horizontally stretch the browser pane to put the table out of "card" mode and into "table" mode.

To reproduce

1. Make a table with a large number of columns and fairly long columns names.
2. Note that the column names overlap the first row...see screenshot below.

[m-preiner]

Update: It looks this issue does not occur with previous releases. For example, mui-datatables@2.0.0 does not have this problem.

[gabrielliwerant]

Thank you for pointing this out. I know what the issue is and will have a fix shortly.

[gabrielliwerant]

Should be resolved in 2.6.2.

[m-preiner]

Awesome, thanks!

…

On Wed, Jul 10, 2019 at 5:29 AM Gabriel Liwerant ***@***.***> wrote: Should be resolved in 2.6.2. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#754?email_source=notifications&email_token=AJZOJ6HJEXO65RNBYIBSHULP6VJSDA5CNFSM4H665SG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZSFTLI#issuecomment-509893037>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AJZOJ6BCUXIBITDBIDS5VRDP6VJSDANCNFSM4H665SGQ> .

[hibc]

@gabrielliwerant
Hello I just updated to v2.6.2.
I didn't encounter overlap issue, but the resizing still doesn't work when I have about 20 columns.
(Resizing works if I have about 10 columns)

When trying to resize a column, the width and height of the column stays the same, but only the column divider bar is moved.

my option is

option = {{
    onRowClick: () => { // do something},
    resizableColumns: true,
    responsive: 'scroll'
}}

[gabrielliwerant]

@hibc Hello. That sounds like a different issue. The one mentioned here initially has been resolved as far as I can tell.

If you like, you can open up a new ticket for your issue with steps to reproduce, and we can discuss it there. My feeling though is that you're running into limitations due to lack of remaining screen width. I don't know if that situation has been addressed as far as resizable columns yet. We'd need to determine which rows get width stolen to compensate, and there are other issues with resizable columns such as lack of minimum width which is also probably contributing.