Krinkle has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/184561

Change subject: mw.loader: Remove support for IE 6
......................................................................

mw.loader: Remove support for IE 6

The client loader doesn't run in IE 6.

Since MediaWiki 1.24, the s tartup module provides IE 6 a
javascriptless environment.

Change-Id: I16628e21d6a89a31dc65baf4a22c6fb31dde2535
---
M resources/src/mediawiki/mediawiki.js
1 file changed, 4 insertions(+), 16 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/61/184561/1

diff --git a/resources/src/mediawiki/mediawiki.js 
b/resources/src/mediawiki/mediawiki.js
index e0e2963..5753e34 100644
--- a/resources/src/mediawiki/mediawiki.js
+++ b/resources/src/mediawiki/mediawiki.js
@@ -732,26 +732,15 @@
                         *
                         * @private
                         * @param {string} text CSS text
-                        * @param {HTMLElement|jQuery} [nextnode=document.head] 
The element where the style tag should be
-                        *  inserted before. Otherwise it will be appended to 
`<head>`.
+                        * @param {HTMLElement|jQuery} [nextnode=document.head] 
The element where the style tag
+                        *  should be inserted before. Otherwise it will be 
appended to `<head>`.
                         * @return {HTMLElement} Reference to the created 
`<style>` element.
                         */
                        function newStyleTag( text, nextnode ) {
                                var s = document.createElement( 'style' );
                                // Insert into document before setting cssText 
(bug 33305)
                                if ( nextnode ) {
-                                       // Must be inserted with native 
insertBefore, not $.fn.before.
-                                       // When using jQuery to insert it, like 
$nextnode.before( s ),
-                                       // then IE6 will throw "Access is 
denied" when trying to append
-                                       // to .cssText later. Some kind of 
weird security measure.
-                                       // 
http://stackoverflow.com/q/12586482/319266
-                                       // Works: jsfiddle.net/zJzMy/1
-                                       // Fails: jsfiddle.net/uJTQz
-                                       // Works again: 
http://jsfiddle.net/Azr4w/ (diff: the next 3 lines)
-                                       if ( nextnode.jquery ) {
-                                               nextnode = nextnode.get( 0 );
-                                       }
-                                       nextnode.parentNode.insertBefore( s, 
nextnode );
+                                       $( nextnode ).before( s );
                                } else {
                                        document.getElementsByTagName( 'head' 
)[0].appendChild( s );
                                }
@@ -1421,8 +1410,7 @@
                                        currReqBase
                                );
                                request = sortQuery( request );
-                               // Append &* to avoid triggering the IE6 
extension check
-                               addScript( sourceLoadScript + '?' + $.param( 
request ) + '&*', null, async );
+                               addScript( sourceLoadScript + '?' + $.param( 
request ), null, async );
                        }
 
                        /**

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I16628e21d6a89a31dc65baf4a22c6fb31dde2535
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Krinkle <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to