rooty/DataTables
1
0
Fork 0
mirror of https://github.com/DataTables/DataTables.git synced 2025-03-21 13:29:04 +01:00
Code Issues Releases Activity

New: Support for box-sizing: border-box;

Browse Source 
- Setting `box-sizing: border-box;` for table cells would cause
  DataTables to incorrectly calculate the size of the element when
  applying the scrolling draw. This is because jQuery's $().width()
  always returns the content width (taking into account box-sizing).

- One possible fix was to detect the box model used and switch between
  width() and outerWidth(), but a much better fix is to use
  $().css('width') as this does take into account the box-model and
  allows DataTables to draw the scrolling table columns correctly,
  regardless of the box model. It should actually also improve
  performance, since jQuery doesn't need to look the box model up
  itself.

- This fixes issue #157
This commit is contained in:
Allan Jardine 2013-06-21 09:45:29 +01:00
parent 370a1e7f7a
commit 95be4568eb
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
media/src/core/core.scrolling.js
View File

@ -318,7 +318,7 @@ function _fnScrollDraw ( o )
// Read all widths in next pass. Forces layout only once because we do not change
// any DOM properties.
_fnApplyToChildren( function(nSizer) {
aApplied.push( _fnStringToCss( $(nSizer).width() ) );
aApplied.push( _fnStringToCss( $(nSizer).css('width') ) );
}, anHeadSizers );
// Apply all widths in final pass. Invalidates layout only once because we do not
@ -335,7 +335,7 @@ function _fnScrollDraw ( o )
_fnApplyToChildren( zeroOut, anFootSizers );
_fnApplyToChildren( function(nSizer) {
aAppliedFooter.push( _fnStringToCss( $(nSizer).width() ) );
aAppliedFooter.push( _fnStringToCss( $(nSizer).css('width') ) );
}, anFootSizers );
_fnApplyToChildren( function(nToSize, i) {