Skip to content

Commit

Permalink
Remove IE11 warning message and related files
Browse files Browse the repository at this point in the history
- resolves #7993
  • Loading branch information
gianlucadecola authored and lb- committed Feb 22, 2022
1 parent b8405e9 commit aaee9b8
Show file tree
Hide file tree
Showing 10 changed files with 9 additions and 64 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.txt
Expand Up @@ -12,6 +12,7 @@ Changelog
* Add black configuration and reformat code using it (Dan Braghis)
* Remove UI code for legacy browser support: polyfills, IE11 workarounds, Modernizr (Thibaud Colas)
* Remove redirect auto-creation recipe from documentation as this feature is now supported in Wagtail core (Andy Babic)
* Remove IE11 warnings (Gianluca De Cola)
* Fix: When using `simple_translations` ensure that the user is redirected to the page edit view when submitting for a single locale (Mitchel Cabuloy)
* Fix: When previewing unsaved changes to `Form` pages, ensure that all added fields are correctly shown in the preview (Joshua Munn)

Expand Down
1 change: 1 addition & 0 deletions CONTRIBUTORS.rst
Expand Up @@ -562,6 +562,7 @@ Contributors
* Rizwan Mansuri
* Dennis McGregor
* Joshua Munn
* Gianluca De Cola

Translators
===========
Expand Down
2 changes: 0 additions & 2 deletions client/src/config/wagtailConfig.js
Expand Up @@ -5,8 +5,6 @@ export const ADMIN_URLS = global.wagtailConfig.ADMIN_URLS;
// Maximum number of pages to load inside the explorer menu.
export const MAX_EXPLORER_PAGES = 200;

export const IS_IE11 = !global.ActiveXObject && 'ActiveXObject' in global;

export const LOCALE_NAMES = new Map();

/* eslint-disable-next-line camelcase */
Expand Down
9 changes: 1 addition & 8 deletions client/src/entrypoints/admin/wagtailadmin.js
@@ -1,12 +1,6 @@
import React from 'react';
import ReactDOM from 'react-dom';
import {
Icon,
Portal,
initFocusOutline,
initIE11Warning,
initUpgradeNotification,
} from '../..';
import { Icon, Portal, initFocusOutline, initUpgradeNotification } from '../..';

if (process.env.NODE_ENV === 'development') {
// Run react-axe in development only, so it does not affect performance
Expand All @@ -27,6 +21,5 @@ window.wagtail.components = {
*/
document.addEventListener('DOMContentLoaded', () => {
initFocusOutline();
initIE11Warning();
initUpgradeNotification();
});
21 changes: 0 additions & 21 deletions client/src/includes/initIE11Warning.js

This file was deleted.

22 changes: 0 additions & 22 deletions client/src/includes/initIE11Warning.test.js

This file was deleted.

2 changes: 0 additions & 2 deletions client/src/index.ts
Expand Up @@ -10,7 +10,6 @@ import Portal from './components/Portal/Portal';
import PublicationStatus from './components/PublicationStatus/PublicationStatus';
import Transition from './components/Transition/Transition';
import { initFocusOutline } from './utils/focus';
import { initIE11Warning } from './includes/initIE11Warning';
import { initUpgradeNotification } from './components/UpgradeNotification';

export {
Expand All @@ -21,6 +20,5 @@ export {
PublicationStatus,
Transition,
initFocusOutline,
initIE11Warning,
initUpgradeNotification,
};
2 changes: 1 addition & 1 deletion docs/editor_manual/browser_issues.rst
Expand Up @@ -20,7 +20,7 @@ Wagtail has officially removed support for the legacy Internet Explorer browser
* In Wagtail 2.11 (LTS), there was be a warning message displayed on the Wagtail dashboard for IE11 users with administrator role.
* In Wagtail 2.12, the message was displayed to all users regardless of their role.
* In Wagtail 2.13 and 2.14, the message was displayed at the top of all pages.
* Wagtail will no longer support IE11 as of version 2.15 and beyond.
* Wagtail will no longer support IE11 as of version 2.15 and beyond, for any releases after 2.16 there will no longer be a warning to users.

If this affects you or your organisation, consider which alternative browsers you may be able to use.
Wagtail is fully compatible with Microsoft Edge, Microsoft’s replacement for Internet Explorer. You may consider using its `IE mode <https://docs.microsoft.com/en-us/deployedge/edge-ie-mode>`_ to keep access to IE11-only sites, while other sites and apps like Wagtail can use modern browser capabilities.
Expand Down
5 changes: 5 additions & 0 deletions docs/releases/2.17.md
Expand Up @@ -23,6 +23,7 @@ Here are other changes related to the redesign:
* Add black configuration and reformat code using it (Dan Braghis)
* Remove UI code for legacy browser support: polyfills, IE11 workarounds, Modernizr (Thibaud Colas)
* Remove redirect auto-creation recipe from documentation as this feature is now supported in Wagtail core (Andy Babic)
* Remove IE11 warnings (Gianluca De Cola)


### Bug fixes
Expand All @@ -34,3 +35,7 @@ Here are other changes related to the redesign:

## Upgrade considerations

## Removed warning in Internet Explorer (IE11)

* IE11 support was officially dropped in Wagtail 2.15, as of this release there will no longer be a warning shown to users of this browser.
* Wagtail is fully compatible with Microsoft Edge, Microsoft’s replacement for Internet Explorer. You may consider using its `IE mode <https://docs.microsoft.com/en-us/deployedge/edge-ie-mode>`_ to keep access to IE11-only sites, while other sites and apps like Wagtail can leverage modern browser capabilities.
8 changes: 0 additions & 8 deletions wagtail/admin/templates/wagtailadmin/skeleton.html
Expand Up @@ -76,14 +76,6 @@
{% endblocktrans %}
</noscript>

{% block ie11warning %}
<div data-ie11-warning hidden>
<div class="capabilitymessage">
{% blocktrans with url="https://docs.wagtail.org/en/stable/editor_manual/browser_issues.html#ie11" %}Wagtail has removed support for IE11. <a href="{{ url }}">Learn more about IE11 alternatives</a>.{% endblocktrans %}
</div>
</div>
{% endblock %}

{% block js %}{% endblock %}

<a class="skiplink button" href="#main" data-skiplink>{% trans 'Skip to main content' %}</a>
Expand Down

0 comments on commit aaee9b8

Please sign in to comment.