Merge "Hide ULS trigger from unsupported browsers"

This commit is contained in:
jenkins-bot
2013-06-04 10:34:15 +00:00
committed by Gerrit Code Review
2 changed files with 9 additions and 5 deletions

View File

@@ -1,4 +1,4 @@
/* Don't show it to users who disabled JS */ /* Don't show it to users who disabled JS */
.client-nojs #pt-uls { .client-nojs #pt-uls {
visibility: hidden; display: none;
} }

View File

@@ -112,10 +112,13 @@
* @return boolean * @return boolean
*/ */
mw.uls.isBrowserSupported = function () { mw.uls.isBrowserSupported = function () {
// boxModel is not supported in IE versions under 8. // Blacklist Grade B browsers IE 6, 7 and IE60-IE79
// We are checking for boxModel here because without that features var ua = navigator.userAgent;
// our grid system fails and it makes the ULS non-functional. if ( /MSIE [67]/i.test( ua ) ) {
return $.support.boxModel; return false;
}
return true;
}; };
/** /**
@@ -151,6 +154,7 @@
$( document ).ready( function () { $( document ).ready( function () {
if ( !mw.uls.isBrowserSupported() ) { if ( !mw.uls.isBrowserSupported() ) {
$( '#pt-uls' ).hide();
return; return;
} }