Index: branches/wmf/1.18wmf1/extensions/ArticleFeedback/modules/jquery.articleFeedback/jquery.articleFeedback.js |
— | — | @@ -3,12 +3,6 @@ |
4 | 4 | */ |
5 | 5 | |
6 | 6 | ( function( $ ) { |
7 | | -// HACK: This code is broken in IE7 and below, so if this is IE < 8, bail now |
8 | | -var browser = $.client.profile(); |
9 | | -if ( browser.name === 'msie' && browser.versionNumber < 8 ) { |
10 | | - return; |
11 | | -} |
12 | | - |
13 | 7 | // Only track users who have been assigned to the tracking group |
14 | 8 | var tracked = 'track' === mw.user.bucket( |
15 | 9 | 'ext.articleFeedback-tracking', mw.config.get( 'wgArticleFeedbackTracking' ) |
Index: branches/wmf/1.18wmf1/extensions/ArticleFeedback/modules/ext.articleFeedback/ext.articleFeedback.startup.js |
— | — | @@ -3,6 +3,12 @@ |
4 | 4 | */ |
5 | 5 | |
6 | 6 | jQuery( function( $ ) { |
| 7 | + // HACK: This code is broken in IE7 and below, so if this is IE < 8, bail now |
| 8 | + var browser = $.client.profile(); |
| 9 | + if ( browser.name === 'msie' && browser.versionNumber < 8 ) { |
| 10 | + return; |
| 11 | + } |
| 12 | + |
7 | 13 | // Load check, is this page ArticleFeedback-enabled ? |
8 | 14 | // Keep in sync with ApiArticleFeedback.php |
9 | 15 | if ( |