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

Update dataTables.fixedHeader.js #2

Closed
wants to merge 1 commit into from
Closed

Update dataTables.fixedHeader.js #2

wants to merge 1 commit into from

Conversation

delta1186
Copy link

@delta1186 delta1186 commented Jan 8, 2019

Use the inline css width property if it exists as the new column width due to the width() function not always returning the correct value.

I am guessing if you accept this request, then you can apply it to other packages if needed and also update the version and minimize.

Use the inline css width property if it exists as the new column width due to the width() function not always returning the correct value.
@DataTables
Copy link
Collaborator

This is repo to send pull results please. The repo here is just an automatically built derivative of it.

Also, could you send me a test case showing the patch fixing the issue at hand? The CSS width can be overridden by the browser, for example if the content of the cell won't fit into the given width and can't be broken down. Using the actual rendered width should be more accurate.

@delta1186 delta1186 closed this Jan 9, 2019
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.

2 participants