Esanders has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/346815 )

Change subject: Remove more IE9 hacks
......................................................................

Remove more IE9 hacks

Change-Id: I0811ebd250ec8c87713440cbe4d8fbfab20a5aa8
---
M src/ce/styles/nodes/ve.ce.AlienNode.css
M src/ce/styles/nodes/ve.ce.TableNode.css
M src/ce/styles/ve.ce.Surface.css
M src/ui/styles/elements/ve.ui.DiffElement.css
M src/ve.utils.js
5 files changed, 0 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/VisualEditor/VisualEditor 
refs/changes/15/346815/1

diff --git a/src/ce/styles/nodes/ve.ce.AlienNode.css 
b/src/ce/styles/nodes/ve.ce.AlienNode.css
index 0d246e4..24a2b98 100644
--- a/src/ce/styles/nodes/ve.ce.AlienNode.css
+++ b/src/ce/styles/nodes/ve.ce.AlienNode.css
@@ -6,7 +6,6 @@
 
 .ve-ce-alienNode-highlights .ve-ce-focusableNode-highlight {
        background-color: #95d14f;
-       /* TODO: Support IE9 with fallback T135497 */
        /* stylelint-disable no-unsupported-browser-features */
        background-image: -webkit-repeating-linear-gradient( -45deg, #fff 0, 
#fff 5px, #95d14f 5px, #95d14f 10px );
        background-image: -moz-repeating-linear-gradient( -45deg, #fff 0, #fff 
5px, #95d14f 5px, #95d14f 10px );
@@ -17,7 +16,6 @@
 }
 
 .ve-ce-surface-highlights-focused .ve-ce-alienNode-highlights 
.ve-ce-focusableNode-highlight {
-       /* TODO: Support IE9 with fallback T135497 */
        /* stylelint-disable no-unsupported-browser-features */
        background-image: -webkit-repeating-linear-gradient( -45deg, #6da9f7 0, 
#6da9f7 5px, #95d14f 5px, #95d14f 10px );
        background-image: -moz-repeating-linear-gradient( -45deg, #6da9f7 0, 
#6da9f7 5px, #95d14f 5px, #95d14f 10px );
diff --git a/src/ce/styles/nodes/ve.ce.TableNode.css 
b/src/ce/styles/nodes/ve.ce.TableNode.css
index 06ea1b6..e037dae 100644
--- a/src/ce/styles/nodes/ve.ce.TableNode.css
+++ b/src/ce/styles/nodes/ve.ce.TableNode.css
@@ -50,7 +50,6 @@
 }
 
 .ve-ce-tableNodeOverlay-selection-box-notEditable {
-       /* TODO: Support IE9 with fallback T135497 */
        /* stylelint-disable no-unsupported-browser-features */
        background-image: -webkit-repeating-linear-gradient( -45deg, 
transparent 0 transparent 5px, #95d14f 5px, #95d14f 10px );
        background-image: -moz-repeating-linear-gradient( -45deg, transparent 
0, transparent 5px, #95d14f 5px, #95d14f 10px );
diff --git a/src/ce/styles/ve.ce.Surface.css b/src/ce/styles/ve.ce.Surface.css
index eb078e2..9f93e77 100644
--- a/src/ce/styles/ve.ce.Surface.css
+++ b/src/ce/styles/ve.ce.Surface.css
@@ -20,7 +20,6 @@
        top: 0;
        left: 0;
        opacity: 0.5;
-       /* TODO: Support IE9 with JS mouse events */
        pointer-events: none; /* stylelint-disable-line 
no-unsupported-browser-features */
 }
 
diff --git a/src/ui/styles/elements/ve.ui.DiffElement.css 
b/src/ui/styles/elements/ve.ui.DiffElement.css
index eb032f0..9e8edb7 100644
--- a/src/ui/styles/elements/ve.ui.DiffElement.css
+++ b/src/ui/styles/elements/ve.ui.DiffElement.css
@@ -47,7 +47,6 @@
        outline: 3px solid #6da9f7;
        padding: 2px;
        margin: -2px 0 0 -2px;
-       /* TODO: Support IE9 with JS mouse events */
        pointer-events: none; /* stylelint-disable-line 
no-unsupported-browser-features */
 }
 
diff --git a/src/ve.utils.js b/src/ve.utils.js
index 2a9071e..029ba1a 100644
--- a/src/ve.utils.js
+++ b/src/ve.utils.js
@@ -839,7 +839,6 @@
        // value is not actually a Document, but something which behaves just 
like an empty regular
        // object...), so we're detecting that and using the innerHTML hack 
described above.
 
-       // Support: IE9
        // Support: Firefox 20
        // Support: Opera 12
 
@@ -857,10 +856,6 @@
        newDocument.close();
        // Detach the iframe
        iframe.parentNode.removeChild( iframe );
-       // Support: IE9
-       // Prevent garbage collection of iframe as long as newDocument exists, 
as destroying
-       // the original iframe makes access to the document impossible in IE9
-       newDocument.originalIframe = iframe;
 
        if ( !newDocument.documentElement || 
newDocument.documentElement.cloneNode( false ) === undefined ) {
                // Surprise! The document is not a document! Only happens on 
Opera.

-- 
To view, visit https://gerrit.wikimedia.org/r/346815
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0811ebd250ec8c87713440cbe4d8fbfab20a5aa8
Gerrit-PatchSet: 1
Gerrit-Project: VisualEditor/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Esanders <esand...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to