Skip to content
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

Unable to view Email in IE 11 #191

Closed
siteshackinc opened this issue Jun 24, 2015 · 7 comments
Closed

Unable to view Email in IE 11 #191

siteshackinc opened this issue Jun 24, 2015 · 7 comments

Comments

@siteshackinc
Copy link

When going to read mail, After selecting the user to see a list of emails, the page is blank.

Confirmed in IE 11 on Windows 8.1

@iliajie
Copy link
Collaborator

iliajie commented Jun 24, 2015

I will be able to test it by the end of the month, when I get to my debug server.

@jamesmacwhite
Copy link

Its not just limited to IE 11, its versions 9 and 10 as well. I guess maybe older versions of IE also, but supporting those is debatable these days!

The cause of this is a jQuery exception in default.min.js:

Object doesn't support property or method 'remove'

webmin-readmail-js-ie

The quick fix I did was replace a.remove (two occurrences) with $(this).remove. If I'm honest I haven't had much time to test it or debug it further, but hope the further details are helpful to this issue. There are more elegant ways to solve it.

@iliajie
Copy link
Collaborator

iliajie commented Jun 27, 2015

This is helpful, thanks pal! I will fix it upon next release by the end of the month! ;)

@iliajie
Copy link
Collaborator

iliajie commented Jun 27, 2015

Ahh, ok! Please test the following - clear browsers cache after replacing default.min.js

Does it work now?

https://rostovtsev.ru/pub/src/authentic/13.05-patches/issue191/default.min.js

@jamesmacwhite
Copy link

The patch is working for me in IE 11.

@iliajie
Copy link
Collaborator

iliajie commented Jun 29, 2015

:) good to hear.

@iliajie
Copy link
Collaborator

iliajie commented Jun 30, 2015

Will add it to version 13.10 very soon.

@iliajie iliajie closed this as completed Jun 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants