Remove is_khtml again
authorAryeh Gregor <simetrical@users.mediawiki.org>
Thu, 16 Jul 2009 02:58:30 +0000 (02:58 +0000)
committerAryeh Gregor <simetrical@users.mediawiki.org>
Thu, 16 Jul 2009 02:58:30 +0000 (02:58 +0000)
This variable is completely misnamed, since it matches WebKit as well.
It's also almost certainly completely useless, since the probability of
some bug existing in all versions of KHTML/WebKit past, present, and
future is approximately zero.  Use capability testing if at all
possible; and if it's not possible, then at least check "browser X
version <= y".  *Never* check for just "browser X"!  That will always
break if a new, fixed version comes out.

skins/common/wikibits.js

index 595d66d..8702404 100644 (file)
@@ -10,8 +10,6 @@ if (webkit_match) {
        var is_safari_win = is_safari && clientPC.indexOf('windows') != -1;
        var webkit_version = parseInt(webkit_match[1]);
 }
-var is_khtml = navigator.vendor == 'KDE' ||
-       ( document.childNodes && !document.all && !navigator.taintEnabled );
 // For accesskeys; note that FF3+ is included here!
 var is_ff2 = /firefox\/[2-9]|minefield\/3/.test( clientPC );
 var ff2_bugs = /firefox\/2/.test( clientPC );