rooty/DataTables
1
0
Fork 0
mirror of https://github.com/DataTables/DataTables.git synced 2025-02-19 17:54:14 +01:00
Code Issues Releases Activity

Fix: Sort wrapper on the header wasn't being removed on fnDestroy - 4617

Browse Source 
New: Span elements in the header when using JUI themes, now have the class of DataTables_sort_icon applied to them automatically
This commit is contained in:
Allan Jardine 2011-04-23 08:40:34 +01:00
parent d419c4ef66
commit 8481826407
2 changed files with 14 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
examples/basic_init/themes.html
View File

@ -18,6 +18,7 @@
"bJQueryUI": true,
"sPaginationType": "full_numbers"
});
oTable.fnDestroy();
} );
</script>
</head>

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

@ -183,6 +183,7 @@
"sSortJUIAscAllowed": "",
"sSortJUIDescAllowed": "",
"sSortJUIWrapper": "",
"sSortIcon": "",
/* Scrolling */
"sScrollWrapper": "dataTables_scroll",
@ -249,6 +250,7 @@
"sSortJUIAscAllowed": "css_right ui-icon ui-icon-carat-1-n",
"sSortJUIDescAllowed": "css_right ui-icon ui-icon-carat-1-s",
"sSortJUIWrapper": "DataTables_sort_wrapper",
"sSortIcon": "DataTables_sort_icon",
/* Scrolling */
"sScrollWrapper": "dataTables_scroll",
@ -2197,7 +2199,14 @@
_oExt.oJUIClasses.sSortableDesc,
_oExt.oJUIClasses.sSortableNone ].join(' ')
);
$('th span', oSettings.nTHead).remove();
$('th span.'+_oExt.oJUIClasses.sSortIcon, oSettings.nTHead).remove();
$('th', oSettings.nTHead).each( function () {
var jqWrapper = $('div.'+_oExt.oJUIClasses.sSortJUIWrapper, this);
var kids = jqWrapper.contents();
$(this).append( kids );
jqWrapper.remove();
} );
}
/* Add the TR elements back into the table in their original order */
@ -2964,7 +2973,9 @@
nDiv.className = oSettings.oClasses.sSortJUIWrapper;
$(nTh).contents().appendTo(nDiv);
nDiv.appendChild( document.createElement('span') );
var nSpan = document.createElement('span');
nSpan.className = oSettings.oClasses.sSortIcon;
nDiv.appendChild( nSpan );
nTh.appendChild( nDiv );
}
}