From f1f200b62e9481995474782e83eab578b14650a3 Mon Sep 17 00:00:00 2001 From: Aryeh Gregor Date: Thu, 16 Jul 2009 02:58:30 +0000 Subject: [PATCH] Remove is_khtml again 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 | 2 -- 1 file changed, 2 deletions(-) diff --git a/skins/common/wikibits.js b/skins/common/wikibits.js index 595d66df70..87024044d2 100644 --- a/skins/common/wikibits.js +++ b/skins/common/wikibits.js @@ -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 ); -- 2.20.1