From 57a536dfcf3a982f8b4a5f234af6a72040228023 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Felix=20St=C3=BCrmer?= <stuermer@weltenwort.de>
Date: Thu, 15 Jun 2017 18:05:43 +0200
Subject: [PATCH 1/2] [Accessibility] Avoid empty th in doc table header row

---
 .../public/doc_table/components/table_header.html  | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/src/ui/public/doc_table/components/table_header.html b/src/ui/public/doc_table/components/table_header.html
index 14e9d9d9708d3..3d14a7e08b9ff 100644
--- a/src/ui/public/doc_table/components/table_header.html
+++ b/src/ui/public/doc_table/components/table_header.html
@@ -1,6 +1,10 @@
 <tr>
-  <th width="1%"></th>
-  <th ng-if="indexPattern.timeFieldName" data-test-subj="docTableHeaderField">
+  <td width="1%"></td>
+  <th
+    ng-if="indexPattern.timeFieldName"
+    data-test-subj="docTableHeaderField"
+    scope="col"
+  >
     <span>Time <button
       id="docTableHeaderFieldSort{{indexPattern.timeFieldName}}"
       tabindex="0"
@@ -13,7 +17,11 @@
     ></button>
     </span>
   </th>
-  <th ng-repeat="name in columns" data-test-subj="docTableHeaderField">
+  <th
+    ng-repeat="name in columns"
+    data-test-subj="docTableHeaderField"
+    scope="col"
+  >
     <span class="table-header-name">
       {{name | shortDots}}
       <button

From f19e266d23dbc52b996919ad202acaa64d6c36b5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Felix=20St=C3=BCrmer?= <stuermer@weltenwort.de>
Date: Fri, 16 Jun 2017 10:10:58 +0200
Subject: [PATCH 2/2] Fix doc table header tests

---
 .../doc_table/__tests__/lib/rows_headers.js   | 20 +++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/ui/public/doc_table/__tests__/lib/rows_headers.js b/src/ui/public/doc_table/__tests__/lib/rows_headers.js
index 02ce596eb034e..0ba965cc8fa33 100644
--- a/src/ui/public/doc_table/__tests__/lib/rows_headers.js
+++ b/src/ui/public/doc_table/__tests__/lib/rows_headers.js
@@ -52,7 +52,7 @@ describe('Doc Table', function () {
 
     it('should create a time column if the timefield is defined', function () {
       const childElems = parentElem.find(elemType);
-      expect(childElems.length).to.be(2);
+      expect(childElems.length).to.be(1);
     });
 
     it('should be able to add and remove columns', function () {
@@ -61,20 +61,20 @@ describe('Doc Table', function () {
       $parentScope.columns = ['bytes'];
       parentElem.scope().$digest();
       childElems = parentElem.find(elemType);
-      expect(childElems.length).to.be(3);
-      expect($(childElems[2]).text()).to.contain('bytes');
+      expect(childElems.length).to.be(2);
+      expect($(childElems[1]).text()).to.contain('bytes');
 
       $parentScope.columns = ['bytes', 'request_body'];
       parentElem.scope().$digest();
       childElems = parentElem.find(elemType);
-      expect(childElems.length).to.be(4);
-      expect($(childElems[3]).text()).to.contain('request_body');
+      expect(childElems.length).to.be(3);
+      expect($(childElems[2]).text()).to.contain('request_body');
 
       $parentScope.columns = ['request_body'];
       parentElem.scope().$digest();
       childElems = parentElem.find(elemType);
-      expect(childElems.length).to.be(3);
-      expect($(childElems[2]).text()).to.contain('request_body');
+      expect(childElems.length).to.be(2);
+      expect($(childElems[1]).text()).to.contain('request_body');
     });
 
     it('should create only the toggle column if there is no timeField', function () {
@@ -82,7 +82,7 @@ describe('Doc Table', function () {
       parentElem.scope().$digest();
 
       const childElems = parentElem.find(elemType);
-      expect(childElems.length).to.be(1);
+      expect(childElems.length).to.be(0);
     });
 
   };
@@ -117,7 +117,7 @@ describe('Doc Table', function () {
     });
 
     describe('adding and removing columns', function () {
-      columnTests('th', $elem);
+      columnTests('[data-test-subj~="docTableHeaderField"]', $elem);
     });
 
     describe('cycleSortOrder function', function () {
@@ -252,7 +252,7 @@ describe('Doc Table', function () {
     });
 
     describe('adding and removing columns', function () {
-      columnTests('td', $elem);
+      columnTests('[data-test-subj~="docTableField"]', $elem);
     });
 
     describe('details row', function () {