ba292272441e8cb0f155d421688711a7cf7280f2 Fix: aria-label for sortin…

…g should be on the element with `role=button`

https://datatables.net/forums/discussion/80509

165ea7f3bfce315ed8a9487286a15b604c0a6eac Examples: Add lang="en" to html tag

Sync to source repo @165ea7f3bfce315ed8a9487286a15b604c0a6eac

datatables.json

@@ -11,5 +11,5 @@
     ],
     "src-repo": "http://github.com/DataTables/DataTablesSrc",
     "last-tag": "2.2.1",
-    "last-sync": "9f7bbc77a204392ce37883057264b4be2ebbfcb4"
+    "last-sync": "165ea7f3bfce315ed8a9487286a15b604c0a6eac"
 }

js/dataTables.js

@@ -13050,16 +13050,16 @@
 						cell.removeAttr('aria-sort');
 					}
 
-					cell.attr('aria-label', orderable
-						? col.ariaTitle + ctx.api.i18n('oAria.orderable' + ariaType)
-						: col.ariaTitle
-					);
-
 					// Make the headers tab-able for keyboard navigation
 					if (orderable) {
 						var orderSpan = cell.find('.dt-column-order');
 
-						orderSpan.attr('role', 'button');
+						orderSpan
+							.attr('role', 'button')
+							.attr('aria-label', orderable
+								? col.ariaTitle + ctx.api.i18n('oAria.orderable' + ariaType)
+								: col.ariaTitle
+							);
 
 						if (tabIndex !== -1) {
 							orderSpan.attr('tabindex', tabIndex);

js/dataTables.mjs

@@ -13015,16 +13015,16 @@ $.extend( true, DataTable.ext.renderer, {
 					cell.removeAttr('aria-sort');
 				}
 
-				cell.attr('aria-label', orderable
-					? col.ariaTitle + ctx.api.i18n('oAria.orderable' + ariaType)
-					: col.ariaTitle
-				);
-
 				// Make the headers tab-able for keyboard navigation
 				if (orderable) {
 					var orderSpan = cell.find('.dt-column-order');
 
-					orderSpan.attr('role', 'button');
+					orderSpan
+						.attr('role', 'button')
+						.attr('aria-label', orderable
+							? col.ariaTitle + ctx.api.i18n('oAria.orderable' + ariaType)
+							: col.ariaTitle
+						);
 
 					if (tabIndex !== -1) {
 						orderSpan.attr('tabindex', tabIndex);