On Wed, May 7, 2008 at 5:36 AM,  <toad at freenetproject.org> wrote:
> Author: toad
>  Date: 2008-05-06 21:36:51 +0000 (Tue, 06 May 2008)
>  New Revision: 19807
>
>  Modified:
>    trunk/freenet/src/freenet/clients/http/WelcomeToadlet.java
>  Log:
>  It's "safari" not "Safari". Do toLowerCase() first when checking.
>  Fixes activelinks still showing up on Safari despite fix that was supposed 
> to prevent this because Safari can't handle it.
>

You should check for konqueror and qtwebkit as well, they share the
same code base. Checking for "KHTML" should be sufficient

Safari:
     Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en) AppleWebKit/XX
(KHTML, like Gecko) Safari/YY
Konqueror:
     Mozilla/5.0 (compatible; Konqueror/4.0; Linux; x86_64)
KHTML/4.0.2 (like Gecko)
     Mozilla/5.0 (compatible; Konqueror/3.5; Windows NT 6.0)
KHTML/3.5.6 (like Gecko)
QtWebKit:
     Mozilla/5.0 (%Platform%; %Security%; %Subplatform%; %Locale%)
AppleWebKit/%WebKitVersion% (KHTML, like Gecko, Safari/419.3)
%AppVersion%

Reply via email to