From 48988ba27135eb5013967a8b43848ae47eccdcd9 Mon Sep 17 00:00:00 2001
From: nfreear <nfreear>
Date: Tue, 28 Nov 2006 15:25:00 +0000
Subject: [PATCH] Fix for MDL-6685 (and MDL-7122 "breadcrumb separator
 missing").

---
 lib/weblib.php | 23 ++++++++++++++++++++---
 1 file changed, 20 insertions(+), 3 deletions(-)

diff --git a/lib/weblib.php b/lib/weblib.php
index 064248bce6b..c196c13bf30 100644
--- a/lib/weblib.php
+++ b/lib/weblib.php
@@ -2670,12 +2670,29 @@ function check_theme_arrows() {
     global $THEME;
     
     if (!isset($THEME->rarrow) and !isset($THEME->larrow)) {
+        // Default, looks good in Win XP/IE 6, Win/Firefox 1.5, Win/Netscape 8...
+        // Also OK in Win 9x/2K/IE 5.x
         $THEME->rarrow = '&#x25BA;';
         $THEME->larrow = '&#x25C4;';
-        if (FALSE !== strpos($_SERVER['HTTP_USER_AGENT'], 'Opera')) {
+        $uagent = $_SERVER['HTTP_USER_AGENT'];
+        if (false !== strpos($uagent, 'Opera')
+            || false !== strpos($uagent, 'Mac')) {
+            // Looks good in Win XP/Mac/Opera 8/9, Mac/Firefox 2, Camino, Safari.
+            // Not broken in Mac/IE 5, Mac/Netscape 7 (?).
             $THEME->rarrow = '&#x25B6;';
             $THEME->larrow = '&#x25C0;';
-       }
+        }
+        elseif (false !== strpos($uagent, 'Konqueror')) {
+            $THEME->rarrow = '&rarr;';
+            $THEME->larrow = '&larr;';
+        }
+        elseif (isset($_SERVER['HTTP_ACCEPT_CHARSET'])
+            && false === stripos($_SERVER['HTTP_ACCEPT_CHARSET'], 'utf-8')) {
+            // (Win/IE 5 doesn't set ACCEPT_CHARSET, but handles Unicode.)
+            // To be safe, non-Unicode browsers!
+            $THEME->rarrow = '&gt;';
+            $THEME->larrow = '&lt;';
+        }
     }
 }
 
@@ -5405,4 +5422,4 @@ function disable_debugging() {
 }
 
 // vim:autoindent:expandtab:shiftwidth=4:tabstop=4:tw=140:
-?>
+?>
\ No newline at end of file