4bc2118f2747e82d9509d4da4db0058b02230870 Update: Moved the tab focus …

…(for keyboard access to ordering) to the ordering icon, rather than the full header cells. This makes a lot more sense as it acts as a button to trigger ordering, and it addresses a spec violation error which dictates that you cannot have `aria-sort` and `tabindex` on the same elements.

DD-3008
https://datatables.net/forums/discussion/79934

Sync to source repo @4bc2118f2747e82d9509d4da4db0058b02230870

datatables.json

@@ -11,5 +11,5 @@
     ],
     "src-repo": "http://github.com/DataTables/DataTablesSrc",
     "last-tag": "2.1.8",
-    "last-sync": "15a28f0db351b32b446a20ab2d4accd6acbd9716"
+    "last-sync": "4bc2118f2747e82d9509d4da4db0058b02230870"
 }

js/dataTables.js

@@ -13044,10 +13044,12 @@
 
 					// Make the headers tab-able for keyboard navigation
 					if (orderable) {
-						cell.find('.dt-column-title').attr('role', 'button');
+						var orderSpan = cell.find('.dt-column-order');
+
+						orderSpan.attr('role', 'button');
 
 						if (tabIndex !== -1) {
-							cell.attr('tabindex', 0);
+							orderSpan.attr('tabindex', tabIndex);
 						}
 					}
 				} );

js/dataTables.mjs

@@ -12991,10 +12991,12 @@ $.extend( true, DataTable.ext.renderer, {
 
 				// Make the headers tab-able for keyboard navigation
 				if (orderable) {
-					cell.find('.dt-column-title').attr('role', 'button');
+					var orderSpan = cell.find('.dt-column-order');
+
+					orderSpan.attr('role', 'button');
 
 					if (tabIndex !== -1) {
-						cell.attr('tabindex', 0);
+						orderSpan.attr('tabindex', tabIndex);
 					}
 				}
 			} );