New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Popup Open creates exception in IE with JQM 1.4.5 if document.activeElement is null #7968

Closed
salluvada opened this Issue Feb 19, 2015 · 4 comments

Comments

Projects
None yet
2 participants
@salluvada

salluvada commented Feb 19, 2015

Browser : IE
jQuery Mobile version : 1.4.5

Issue : In IE browser, when a popup is being opened, if the document.activeElement is null, the _safelyBlur method is throwing an exception. So if you haven't clicked on any control or tabbed to any control on a html page, after the page is launched, the document.activeElement is null and due to this the exception is raised. How ever if you click/touch anywhere on the body, the document.activeElement is set to a value and this exception is not raised.

Below is the code from jsbin. Donot run it in jsbin as you cannot reproduce this error. You need to simply copy the html, put it in a .html file and place it in a virtual directory in iis/any other webserver and load it from there.

http://jsbin.com/nivozeniwe/1/watch?html,output

Screen Shot: https://onedrive.live.com/redir?resid=F5B706029F07E38E!6225&authkey=!AEg6QqUjyEogrDQ&v=3&ithint=photo%2cjpg

@gabrielschulhof

This comment has been minimized.

Show comment
Hide comment
@gabrielschulhof

gabrielschulhof Feb 19, 2015

Contributor

@salluvada what version of IE? Also, can you please try with the latest code from master? http://jsbin.com/xiyete/1/

Contributor

gabrielschulhof commented Feb 19, 2015

@salluvada what version of IE? Also, can you please try with the latest code from master? http://jsbin.com/xiyete/1/

@salluvada

This comment has been minimized.

Show comment
Hide comment
@salluvada

salluvada Feb 19, 2015

The latest code you gave me is also giving the same exception.

The issue is occurring in IE 11,10,9,8 as far as I tested. This issue doesn't occur in Chrome because by default in Chrome, the document.activeElement is set to body.

salluvada commented Feb 19, 2015

The latest code you gave me is also giving the same exception.

The issue is occurring in IE 11,10,9,8 as far as I tested. This issue doesn't occur in Chrome because by default in Chrome, the document.activeElement is set to body.

@gabrielschulhof

This comment has been minimized.

Show comment
Hide comment
@gabrielschulhof

gabrielschulhof Feb 19, 2015

Contributor

OK, I see it now. Reproduced.

Contributor

gabrielschulhof commented Feb 19, 2015

OK, I see it now. Reproduced.

@gabrielschulhof gabrielschulhof added this to the 1.5.0 milestone Feb 19, 2015

@gabrielschulhof gabrielschulhof self-assigned this Feb 19, 2015

@gabrielschulhof

This comment has been minimized.

Show comment
Hide comment
@gabrielschulhof

gabrielschulhof Feb 19, 2015

Contributor

This will be fixed once the implementation of $.mobile.safelyBlur() lands, fixing this and #1560.

Contributor

gabrielschulhof commented Feb 19, 2015

This will be fixed once the implementation of $.mobile.safelyBlur() lands, fixing this and #1560.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment