TableData is focusable element for now (closes #901) (#909)

DevExpress · Nov 3, 2016 · 8b8dc42 · 8b8dc42
1 parent b2824f7
commit 8b8dc42
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 0 deletions.
diff --git a/src/client/utils/dom.js b/src/client/utils/dom.js
@@ -489,6 +489,9 @@ export function isElementFocusable (el) {
                matches(el, 'a[href]');
     }
 
+    if (isTableDataElement(el) && isIE)
+        return true;
+
     return matches(el, getFocusableSelector()) || tabIndex !== null;
 }
 
@@ -637,6 +640,10 @@ export function isTableElement (el) {
     return getTagName(el) === 'table';
 }
 
+export function isTableDataElement (el) {
+    return getTagName(el) === 'td';
+}
+
 export function matches (el, selector) {
     if (!el)
         return false;

diff --git a/test/client/data/is-focusable/iframe.html b/test/client/data/is-focusable/iframe.html
@@ -42,5 +42,14 @@
 <input type="button" value="button input" tabindex="-1" class="expected"/>
 
 <iframe class="expected"></iframe>
+<table>
+    <tbody>
+    <tr>
+        <td class="expected">
+            Table data
+        </td>
+    </tr>
+    </tbody>
+</table>
 </body>
 </html>
diff --git a/test/client/fixtures/utils/dom-test.js b/test/client/fixtures/utils/dom-test.js
@@ -603,6 +603,11 @@ asyncTest('isElementFocusable', function () {
                     return !domUtils.isOptionElement(el);
                 });
             }
+            else {
+                expectedFocusedElements = expectedFocusedElements.filter(function (el) {
+                    return !domUtils.isTableDataElement(el);
+                });
+            }
 
             for (var i = 0; i < allElements.length; i++) {
                 if (domUtils.isElementFocusable(allElements[i]))