test.aria.utils.cfgframe.AriaWindowTest failing on IE 8 #974

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Member

divdavem commented Feb 13, 2014

Commit 2c1141e makes the following test fail on IE 8: test.aria.utils.cfgframe.AriaWindowTest. It only happens on a true IE 8 (not on IE 11 in IE 8 mode).

The issue comes from an "Unspecified error" raised when trying to read Aria.$window.document.activeElement. This pull request surrounds that piece of code by a try ... catch.

@divdavem divdavem added a commit to divdavem/ariatemplates that referenced this pull request Feb 13, 2014

@divdavem divdavem fix #974 test.aria.utils.cfgframe.AriaWindowTest failing on IE 8
Commit 2c1141e makes the following
test fail on IE 8: test.aria.utils.cfgframe.AriaWindowTest.
It only happens on a true IE 8 (not on IE 11 in IE 8 mode).

The issue comes from an "Unspecified error" raised when trying to read
Aria.$window.document.activeElement. This commit surrounds that piece
of code by a try ... catch.
f3a20e3
@divdavem divdavem fix #974 test.aria.utils.cfgframe.AriaWindowTest failing on IE 8
Commit 2c1141e makes the following
test fail on IE 8: test.aria.utils.cfgframe.AriaWindowTest.
It only happens on a true IE 8 (not on IE 11 in IE 8 mode).

The issue comes from an "Unspecified error" raised when trying to read
Aria.$window.document.activeElement. This commit surrounds that piece
of code by a try ... catch.
eb58c96
Collaborator

flongo commented Feb 13, 2014

eb58c96 looks fine to me. It can be integrated!

divdavem closed this in eb58c96 Feb 13, 2014

flongo added this to the 1.4.16 milestone Feb 14, 2014

flongo self-assigned this Feb 14, 2014

@carlo-mr carlo-mr added a commit to carlo-mr/ariatemplates that referenced this pull request Mar 3, 2014

@divdavem @carlo-mr divdavem + carlo-mr fix #974 test.aria.utils.cfgframe.AriaWindowTest failing on IE 8
Commit 2c1141e makes the following
test fail on IE 8: test.aria.utils.cfgframe.AriaWindowTest.
It only happens on a true IE 8 (not on IE 11 in IE 8 mode).

The issue comes from an "Unspecified error" raised when trying to read
Aria.$window.document.activeElement. This commit surrounds that piece
of code by a try ... catch.
f5f9926
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment