#18888 closed defect (fixed)
BackgroundIframe not enabled for IE11/Win10
| Reported by: | Earl Hood | Owned by: | |
|---|---|---|---|
| Priority: | undecided | Milestone: | 1.12 |
| Component: | Dijit | Version: | 1.11.2 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description
The BackgroundIframe? is not enabled for IE11 under Windows 10 environments. Because of this, popups, et.al. do not show when popped up over an ActiveX control.
Examining dijit/BackgroundIframe.js, the Windows OS version check needs to be updated to check for "Windows 10.0". For example:
has.add("config-bgIframe",
(has("ie") && !/IEMobile\/10\.0/.test(navigator.userAgent)) || // No iframe on WP8, to match 1.9 behavior
(has("trident") && /Windows NT (?:6\.[01]|10\.0)/.test(navigator.userAgent)));
Workaround for existing (1.11.2) and previous releases of the toolkit can has.add("config-bgIframe") before dijit/BackgroundIframe is loaded to supercede the test for those that need to support IE11/ActiveX/Win10 environments.
Change History (6)
comment:1 Changed 4 years ago by
comment:2 Changed 4 years ago by
| Milestone: | tbd → 1.12 |
|---|
comment:3 Changed 4 years ago by
My testing showed that the BackgroundIFrame was only needed for IE11 on Win7. That's why I wrote the code to limit to that case. Apparently you are seeing cases where it's needed on IE11 on Win10?
comment:4 Changed 4 years ago by
| Owner: | set to Michael Van Sickle <[email protected]…> |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
In c07a4cd/dijit:
Note: See
TracTickets for help on using
tickets.
Windows 8(.1) should also be supported. That would update the has.add() to the following:
has.add("config-bgIframe", (has("ie") && !/IEMobile\/10\.0/.test(navigator.userAgent)) || // No iframe on WP8, to match 1.9 behavior (has("trident") && /Windows NT (?:6\.[0-3]|10\.0)/.test(navigator.userAgent)));