From 213fa0e71e39dff44c109069a030fa28bc564f70 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Niels=20Lyngs=C3=B8?= Date: Thu, 21 Nov 2019 12:14:26 +0100 Subject: [PATCH] return true, cause DocumentFragment is never displayed. (cherry picked from commit ec9c99554ace24b5a1fbeae4a22dbe29cdc62f77) --- .../src/common/services/tabbable.service.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Umbraco.Web.UI.Client/src/common/services/tabbable.service.js b/src/Umbraco.Web.UI.Client/src/common/services/tabbable.service.js index 2f7af50804..feab7860ca 100644 --- a/src/Umbraco.Web.UI.Client/src/common/services/tabbable.service.js +++ b/src/Umbraco.Web.UI.Client/src/common/services/tabbable.service.js @@ -186,7 +186,7 @@ if (!nodeComputedStyle) { if (node instanceof DocumentFragment) { - return false; + return true;// though DocumentFragment dosnt directly have display 'none', we know that it will never be visible, and therefor we return true. (and do not cache this, cause it will change if appended to the DOM) } else { nodeComputedStyle = this.doc.defaultView.getComputedStyle(node); }