rooty/DataTables
1
0
Fork 0
mirror of https://github.com/DataTables/DataTables.git synced 2024-11-29 11:24:10 +01:00
Code Issues Releases Activity

Remove pointless duplication of code - it was doing the same thing, just with different pointers

Browse Source
This commit is contained in:
Allan Jardine 2011-12-14 10:48:15 +00:00
parent 169eb721d2
commit c8c40189a2
2 changed files with 16 additions and 29 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
media/js/jquery.dataTables.js vendored
View File

@ -5122,21 +5122,14 @@
$(_fnGetTrNodes( oSettings )).removeClass( oSettings.asStripeClasses.join(' ') );
if ( !oSettings.bJUI )
$('th', oSettings.nTHead).removeClass( [
oSettings.oClasses.sSortable,
oSettings.oClasses.sSortableAsc,
oSettings.oClasses.sSortableDesc,
oSettings.oClasses.sSortableNone ].join(' ')
);
if ( oSettings.bJUI )
{
$('th', oSettings.nTHead).removeClass( [ DataTable.ext.oStdClasses.sSortable,
DataTable.ext.oStdClasses.sSortableAsc,
DataTable.ext.oStdClasses.sSortableDesc,
DataTable.ext.oStdClasses.sSortableNone ].join(' ')
);
}
else
{
$('th, td', oSettings.nTHead).removeClass( [ DataTable.ext.oStdClasses.sSortable,
DataTable.ext.oJUIClasses.sSortableAsc,
DataTable.ext.oJUIClasses.sSortableDesc,
DataTable.ext.oJUIClasses.sSortableNone ].join(' ')
);
$('th span.'+DataTable.ext.oJUIClasses.sSortIcon
+ ', td span.'+DataTable.ext.oJUIClasses.sSortIcon, oSettings.nTHead).remove();
@ -8416,7 +8409,8 @@
* and filtering) by adding a tabindex attribute to the required elements. This
* allows you to tab through the controls and press the enter key to activate them.
* The tabindex is default 0, meaning that the tab follows the flow of the document.
* You can overrule this using this parameter if you wish.
* You can overrule this using this parameter if you wish. Use a value of -1 to
* disable built-in keyboard navigation.
* @type int
* @default 0
*

21
media/src/api/api.methods.js
View File

@ -416,21 +416,14 @@ this.fnDestroy = function ( bRemove )
$(_fnGetTrNodes( oSettings )).removeClass( oSettings.asStripeClasses.join(' ') );
if ( !oSettings.bJUI )
$('th', oSettings.nTHead).removeClass( [
oSettings.oClasses.sSortable,
oSettings.oClasses.sSortableAsc,
oSettings.oClasses.sSortableDesc,
oSettings.oClasses.sSortableNone ].join(' ')
);
if ( oSettings.bJUI )
{
$('th', oSettings.nTHead).removeClass( [ DataTable.ext.oStdClasses.sSortable,
DataTable.ext.oStdClasses.sSortableAsc,
DataTable.ext.oStdClasses.sSortableDesc,
DataTable.ext.oStdClasses.sSortableNone ].join(' ')
);
}
else
{
$('th, td', oSettings.nTHead).removeClass( [ DataTable.ext.oStdClasses.sSortable,
DataTable.ext.oJUIClasses.sSortableAsc,
DataTable.ext.oJUIClasses.sSortableDesc,
DataTable.ext.oJUIClasses.sSortableNone ].join(' ')
);
$('th span.'+DataTable.ext.oJUIClasses.sSortIcon
+ ', td span.'+DataTable.ext.oJUIClasses.sSortIcon, oSettings.nTHead).remove();