Fix: Destroy wasn't correctly removing bound events (causing a memory…

… leak) Fix: Copy to clipboard condition Merge pull request #35 from flare256/master
DataTables · Oct 23, 2017 · 547e4f2 · 547e4f2
2 parents 16af4df + 8ba51a6
commit 547e4f2
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/js/dataTables.keyTable.js b/js/dataTables.keyTable.js
@@ -290,7 +290,7 @@ $.extend( KeyTable.prototype, {
 		dt.on( 'destroy.keyTable', function () {
 			dt.off( '.keyTable' );
 			$( dt.table().body() ).off( 'click.keyTable', 'th, td' );
-			$( document.body )
+			$( document )
 				.off( 'keydown.keyTable' )
 				.off( 'click.keyTable' );
 		} );
@@ -629,7 +629,7 @@ $.extend( KeyTable.prototype, {
 			return;
 		}
 
-		if ( e.ctrlKey && e.keyCode === 67 ) { // c
+		if ( e.ctrlKey && !e.altKey && e.keyCode === 67 ) { // c
 			this._clipboardCopy();
 			return;
 		}