-
Notifications
You must be signed in to change notification settings - Fork 357
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
Add X-UA-Compatible header #4946
Conversation
in situations where IE11 fails to detect the "document mode" correctly, this header should ensure that IE uses the "edge" mode. (this probably doesn't help in any case where IE11 is set to force IE7 mode) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1648338
Checked commit https://github.com/himdel/manageiq-ui-classic/commit/2304073aa34e9f3dcb0b910644112c7388493b1f with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 |
cc @martinpovolny .. if you have any idea where this could autodetect wrong and test that this helps... |
Add X-UA-Compatible header (cherry picked from commit 123d000) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1648338
Hammer backport details:
|
Add X-UA-Compatible header (cherry picked from commit 123d000) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1653710
Gaprindashvili backport details:
|
in situations where IE11 fails to detect the "document mode" correctly,
this header should ensure that IE uses the "edge" mode.
(this probably doesn't help in any case where IE11 is set to force IE7 mode)
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1648338
But see the comments in the BZ,
I'm almost sure this is useless, because the problem is not on our side. It won't hurt though, and will fix the problem if I'm wrong :)EDIT: confirmed in the BZ that this header will indeed override group policy settings