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

Warning for unsupported browsers #4809

Closed
tdipisa opened this issue Jan 28, 2020 · 3 comments
Closed

Warning for unsupported browsers #4809

tdipisa opened this issue Jan 28, 2020 · 3 comments

Comments

@tdipisa
Copy link
Member

tdipisa commented Jan 28, 2020

Description

Make the webapp able to intercept the unsupported browser type (eg. IE 11) before loading MS to redirect the user to a static page that contains a warning message, something similar to the following:

browser_unsupported

Other useful information

@tdipisa tdipisa added this to the 2020.01.01 milestone Jan 28, 2020
@tdipisa tdipisa changed the title Warning for not supported browsers Warning for unsupported browsers Jan 28, 2020
@tdipisa tdipisa modified the milestones: 2020.01.01, 2020.02.00 Mar 11, 2020
@tdipisa tdipisa assigned MV88 and unassigned allyoucanmap May 15, 2020
@MV88
Copy link
Contributor

MV88 commented May 19, 2020

My proposal is to add this lib that allows us to quickly filter browsers among a non configurable list , which is Chrome, Firefox, Safari, Edge.

Detection will be made at the very top of the application and specifically in the StandardApp component here
if the browser family is not among the 4 mentioned above then a redirect to a specific informative page will be performed

Also the examples will be updated with new redirect functionality or with some mask

@MV88
Copy link
Contributor

MV88 commented May 19, 2020

Another possibility is to detect browser familiy before running everything at index.html level

@tdipisa
Copy link
Member Author

tdipisa commented May 20, 2020

@MV88 @offtherailz if we can do that without adding a new library is better in my opinion. @offtherailz let me know your thoughts.

@MV88 MV88 added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label May 26, 2020
@tdipisa tdipisa removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label May 26, 2020
MV88 added a commit to MV88/MapStore2 that referenced this issue May 29, 2020
@ElenaGallo ElenaGallo self-assigned this Jun 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants