datatables.net 2.1.3 → 2.1.5

package/js/dataTables.mjs

@@ -1,4 +1,4 @@
-/*! DataTables 2.1.3
+/*! DataTables 2.1.5
  * © SpryMedia Ltd - datatables.net/license
  */
 
@@ -1000,7 +1000,8 @@ $.extend( DataTable.ext.classes, {
 		active: 'current',
 		button: 'dt-paging-button',
 		container: 'dt-paging',
-		disabled: 'disabled'
+		disabled: 'disabled',
+		nav: ''
 	}
 } );
 
@@ -2102,7 +2103,11 @@ function _fnColumnSizes ( settings )
 	for (var i=0 ; i<cols.length ; i++) {
 		var width = _fnColumnsSumWidth(settings, [i], false, false);
 
-		cols[i].colEl.css('width', width);
+		// Need to set the min-width, otherwise the browser might try to collapse
+		// it further
+		cols[i].colEl
+			.css('width', width)
+			.css('min-width', width);
 	}
 }
 
@@ -3068,6 +3073,9 @@ function _fnCreateTr ( oSettings, iRow, nTrIn, anTds )
 				_fnWriteCell(nTd, display[i]);
 			}
 
+			// column class
+			_addClass(nTd, oCol.sClass);
+
 			// Visibility - add or remove as required
 			if ( oCol.bVisible && create )
 			{
@@ -3399,7 +3407,6 @@ function _fnDraw( oSettings, ajaxComplete )
 				var td = aoData.anCells[i];
 
 				_addClass(td, _ext.type.className[col.sType]); // auto class
-				_addClass(td, col.sClass); // column class
 				_addClass(td, oSettings.oClasses.tbody.cell); // all cells
 			}
 
@@ -5173,21 +5180,37 @@ function _fnScrollDraw ( settings )
 	// is because of Responsive which might remove `col` elements, knocking the alignment
 	// of the indexes out.
 	if (settings.aiDisplay.length) {
-		// Get the column sizes from the first row in the table
-		var colSizes = table.children('tbody').eq(0).children('tr').eq(0).children('th, td').map(function (vis) {
-			return {
-				idx: _fnVisibleToColumnIndex(settings, vis),
-				width: $(this).outerWidth()
+		// Get the column sizes from the first row in the table. This should really be a
+		// [].find, but it wasn't supported in Chrome until Sept 2015, and DT has 10 year
+		// browser support
+		var firstTr = null;
+
+		for (i=0 ; i<settings.aiDisplay.length ; i++) {
+			var idx = settings.aiDisplay[i];
+			var tr = settings.aoData[idx].nTr;
+
+			if (tr) {
+				firstTr = tr;
+				break;
 			}
-		});
+		}
+
+		if (firstTr) {
+			var colSizes = $(firstTr).children('th, td').map(function (vis) {
+				return {
+					idx: _fnVisibleToColumnIndex(settings, vis),
+					width: $(this).outerWidth()
+				}
+			});
 
-		// Check against what the colgroup > col is set to and correct if needed
-		for (var i=0 ; i<colSizes.length ; i++) {
-			var colEl = settings.aoColumns[ colSizes[i].idx ].colEl[0];
-			var colWidth = colEl.style.width.replace('px', '');
+			// Check against what the colgroup > col is set to and correct if needed
+			for (var i=0 ; i<colSizes.length ; i++) {
+				var colEl = settings.aoColumns[ colSizes[i].idx ].colEl[0];
+				var colWidth = colEl.style.width.replace('px', '');
 
-			if (colWidth !== colSizes[i].width) {
-				colEl.style.width = colSizes[i].width + 'px';
+				if (colWidth !== colSizes[i].width) {
+					colEl.style.width = colSizes[i].width + 'px';
+				}
 			}
 		}
 	}
@@ -5330,7 +5353,11 @@ function _fnCalculateColumnWidths ( settings )
 		var width = _fnColumnsSumWidth( settings, this, true, false );
 
 		if ( width ) {
+			// Need to set the width and min-width, otherwise the browser
+			// will attempt to collapse the table beyond want might have
+			// been specified
 			this.style.width = width;
+			this.style.minWidth = width;
 
 			// For scrollX we need to force the column width otherwise the
 			// browser will collapse it. If this width is smaller than the
@@ -6715,6 +6742,7 @@ _Api = function ( context, data )
 		return new _Api( context, data );
 	}
 
+	var i;
 	var settings = [];
 	var ctxSettings = function ( o ) {
 		var a = _toSettings( o );
@@ -6724,7 +6752,7 @@ _Api = function ( context, data )
 	};
 
 	if ( Array.isArray( context ) ) {
-		for ( var i=0, ien=context.length ; i<ien ; i++ ) {
+		for ( i=0 ; i<context.length ; i++ ) {
 			ctxSettings( context[i] );
 		}
 	}
@@ -6739,7 +6767,16 @@ _Api = function ( context, data )
 
 	// Initial data
 	if ( data ) {
-		this.push.apply(this, data);
+		// Chrome can throw a max stack error if apply is called with
+		// too large an array, but apply is faster.
+		if (data.length < 10000) {
+			this.push.apply(this, data);
+		}
+		else {
+			for (i=0 ; i<data.length ; i++) {
+				this.push(data[i]);
+			}
+		}
 	}
 
 	// selector
@@ -8429,8 +8466,10 @@ var __column_selector = function ( settings, selector, opts )
 			switch( match[2] ) {
 				case 'visIdx':
 				case 'visible':
-					if (match[1]) {
+					// Selector is a column index
+					if (match[1] && match[1].match(/^\d+$/)) {
 						var idx = parseInt( match[1], 10 );
+
 						// Visible index given, convert to column index
 						if ( idx < 0 ) {
 							// Counting from the right
@@ -8443,9 +8482,19 @@ var __column_selector = function ( settings, selector, opts )
 						return [ _fnVisibleToColumnIndex( settings, idx ) ];
 					}
 					
-					// `:visible` on its own
-					return columns.map( function (col, i) {
-						return col.bVisible ? i : null;
+					return columns.map( function (col, idx) {
+						// Not visible, can't match
+						if (! col.bVisible) {
+							return null;
+						}
+
+						// Selector
+						if (match[1]) {
+							return $(nodes[idx]).filter(match[1]).length > 0 ? idx : null;
+						}
+
+						// `:visible` on its own
+						return idx;
 					} );
 
 				case 'name':
@@ -9758,7 +9807,7 @@ _api_register( 'i18n()', function ( token, def, plural ) {
  *  @type string
  *  @default Version number
  */
-DataTable.version = "2.1.3";
+DataTable.version = "2.1.5";
 
 /**
  * Private data store, containing all of the settings objects that are
@@ -12432,7 +12481,7 @@ DataTable.type('string', {
 		pre: function ( a ) {
 			// This is a little complex, but faster than always calling toString,
 			// http://jsperf.com/tostring-v-check
-			return _empty(a) ?
+			return _empty(a) && typeof a !== 'boolean' ?
 				'' :
 				typeof a === 'string' ?
 					a.toLowerCase() :
@@ -12685,6 +12734,12 @@ $.extend( true, DataTable.ext.renderer, {
 					return;               // table, not a nested one
 				}
 
+				var sorting = ctx.sortDetails;
+
+				if (! sorting) {
+					return;
+				}
+
 				var i;
 				var orderClasses = classes.order;
 				var columns = ctx.api.columns( cell );
@@ -12693,7 +12748,6 @@ $.extend( true, DataTable.ext.renderer, {
 				var ariaType = '';
 				var indexes = columns.indexes();
 				var sortDirs = columns.orderable(true).flatten();
-				var sorting = ctx.sortDetails;
 				var orderedColumns = _pluck(sorting, 'col');
 
 				cell
@@ -13061,7 +13115,11 @@ DataTable.feature.register( 'paging', function ( settings, opts ) {
 
 	var host = $('<div/>')
 		.addClass(settings.oClasses.paging.container + (opts.type ? ' paging_' + opts.type : ''))
-		.append('<nav>');
+		.append(
+			$('<nav>')
+				.attr('aria-label', 'pagination')
+				.addClass(settings.oClasses.paging.nav)
+		);
 	var draw = function () {
 		_pagingDraw(settings, host.children(), opts);
 	};

package/package.json

@@ -4,7 +4,7 @@
     "main": "js/dataTables.js",
     "module": "js/dataTables.mjs",
     "types": "./types/types.d.ts",
-    "version": "2.1.3",
+    "version": "2.1.5",
     "files": [
         "js/**/*.js",
         "js/**/*.mjs",