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.

Sign up for GitHub

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

Jump to bottom

[feature] Display error message for IE11 users #44

Open
mxr576 opened this issue Oct 8, 2020 · 3 comments
Open

[feature] Display error message for IE11 users #44

mxr576 opened this issue Oct 8, 2020 · 3 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@mxr576
Copy link
Collaborator

mxr576 commented Oct 8, 2020 •
edited
Loading

Problem

Swagger UI has dropped IE11 support recently. It means that IE 11 users see nothing in place of the Swagger UI. This is a bad user experience.

Suggested solution

Display a configurable error message for IE11 users in place of the Swagger UI which explains that they should use a different browser if they would like to see the API specification.

swagger-api/swagger-editor#2090
@mxr576 mxr576 added enhancement New feature or request help wanted Extra attention is needed labels Oct 8, 2020
@mxr576
Copy link
Collaborator Author

mxr576 commented Oct 8, 2020

We can use a library like jenssegers/agent to detect the browser of the current user.

@mxr576
Copy link
Collaborator Author

mxr576 commented Jan 5, 2022

or just wait until Drupal 10 😁
https://www.drupal.org/node/3199540

@mxr576
Copy link
Collaborator Author

mxr576 commented Aug 29, 2023

just wait until Drupal 9.5 EOL, drop < Drupal 10 support and close this ticket 🍻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mxr576