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

Unsupported Browsers #34

Open
BNNorman opened this issue Jun 22, 2019 · 5 comments
Open

Unsupported Browsers #34

BNNorman opened this issue Jun 22, 2019 · 5 comments
Labels
complexity: medium It's going to require some thought. enhancement New feature or request

Comments

@BNNorman
Copy link

Just tried to access the AQ map using Internet Explorer - all I got was the heading - no map.

If a browser is not supported I think we should display a message to the user.

@sbrl sbrl added enhancement New feature or request complexity: medium It's going to require some thought. labels Jun 22, 2019
@sbrl
Copy link
Collaborator

sbrl commented Jun 22, 2019

Indeed. Internet Explorer is not support by Microsoft (it's replacement is edge), and neither is it supported by me. I don't actually really have access to a Windows machine to test it on.

The primary browsers supported are Firefox (I use Firefox for development - if it doesn't work in another browser try Firefox) and Chrome (incl. derivatives). Safari should also work, but is untested (if not, open an issue).

Displaying such a message may be a bit of a challenge, but not impossible.

@sbrl
Copy link
Collaborator

sbrl commented Jun 23, 2019

Thinking about this, I'll probably need to see what the errors are in the Internet Explorer Developer Tools. I think you open them with either F11 or F12. Only when I know where it goes wrong will I be able to work out how to display a message to old browsers.

It could be it's failing on a syntax error because of new Javascript features, in which case I'll need to work around that using a different technique than I would otherwise.

Without access to a machine to test this on though, I can't be certain.

@sbrl sbrl added the help wanted Extra attention is needed label Jun 23, 2019
@BNNorman
Copy link
Author

This is what I get with IE.

image

The map display works in Edge - never tried it before as I usually use Firefox.

@sbrl
Copy link
Collaborator

sbrl commented Jun 25, 2019

Ah, I see. It is a syntax error then as I suspected. Thanks!

@sbrl sbrl removed the help wanted Extra attention is needed label Jun 25, 2019
@BNNorman
Copy link
Author

Yup

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity: medium It's going to require some thought. enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants