DataTables / Responsive

Responsive extension for DataTables, providing support for complex tables on all device screen sizes
Other
148 stars 87 forks source link

Fix overflow issues #22

Closed ertogs closed 9 years ago

ertogs commented 9 years ago

As mentioned in your code the calculation of the minWidth per cell is not 100% perfect. I ran in some overflow-issues with large tables ( 30+ cols). To keep the good performance and to fix this i added 20px per column. It works perfect in my case. This might be a quirky work-around, but the only bad result might be a greater cell-padding.

In my case this fixes https://github.com/DataTables/Responsive/issues/20

DataTables commented 9 years ago

Hi,

Thanks for the PR. I hope you don't mind, but I'd rather not use magic numbers to fix the problem, preferring instead to try and understand the underlying issue and resolving that (assuming it can be - I'm sure it can!). As such, I won't be pulling this in directly, but others might find it useful until I find time to investigate the issue properly.